-
-
Notifications
You must be signed in to change notification settings - Fork 94
Update stats for BackendHealthCheck in case of doctrine backend is used #325
Conversation
…ine backend is used
@@ -97,7 +97,7 @@ public function handle(MessageInterface $message, EventDispatcherInterface $disp | |||
*/ | |||
public function getStatus() | |||
{ | |||
return new Success('Channel is running (RabbitMQ) and consumers for all queues available.'); | |||
return new Success('Channel is running (Database) and consumers for all queues available.'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what if someone is testing this message?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It will break indeed, but it was a wrong one
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what do others think? cc @sonata-project/contributors
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it was wrong, Imo lets change it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good enough for me, let's merge it then.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kunicmarko20 everything is a BC-break, see my favorite xkcd ;)
Thanks @qsomazzi ! |
@sonata-project/contributors thanks ! |
Changelog
Subject
The goal of this PR is to fix the way data are manipulated in the case of doctrine backend is used. An SQL query fetch the stats but it raise an error.
Before
In the foreach
$data = '-2'
After
In the while
$data = ['state' => -2, 'cnt' => 3]