Ejegg has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/324343

Change subject: Less noisy queue consumer logging
......................................................................

Less noisy queue consumer logging

Bug: T151954
Change-Id: Ifc98c04c5778a7a1a63e084fb0f69a6707a6b1a8
---
M Core/QueueConsumers/BaseQueueConsumer.php
1 file changed, 12 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/SmashPig 
refs/changes/43/324343/1

diff --git a/Core/QueueConsumers/BaseQueueConsumer.php 
b/Core/QueueConsumers/BaseQueueConsumer.php
index 5367c51..fd10420 100644
--- a/Core/QueueConsumers/BaseQueueConsumer.php
+++ b/Core/QueueConsumers/BaseQueueConsumer.php
@@ -101,10 +101,18 @@
                        }
                        $timeOk = $this->timeLimit === 0 || time() <= 
$startTime + $this->timeLimit;
                        $countOk = $this->messageLimit === 0 || $processed < 
$this->messageLimit;
-                       $debugMessage = 'Data is ' . ( $data === null ? '' : 
'not ' ) . 'null, ' .
-                               "time limit ($this->timeLimit) is " . ( $timeOk 
? 'not ' : '' ) . 'elapsed, ' .
-                               "message limit ($this->messageLimit) is " . ( 
$countOk ? 'not ' : '' ) . 'reached.';
-                       Logger::debug( $debugMessage );
+
+                       $debugMessage = null;
+                       if ( $data === null ) {
+                               $debugMessage = 'Queue is empty';
+                       } else if ( !$timeOk ) {
+                               $debugMessage = "Time limit ($this->timeLimit) 
is elapsed";
+                       } else if ( !$countOk ) {
+                               $debugMessage = "Message limit 
($this->messageLimit) is reached";
+                       }
+                       if ( $debugMessage ) {
+                               Logger::debug( $debugMessage );
+                       }
                }
                while( $timeOk && $countOk && $data !== null );
                return $processed;

-- 
To view, visit https://gerrit.wikimedia.org/r/324343
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifc98c04c5778a7a1a63e084fb0f69a6707a6b1a8
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/SmashPig
Gerrit-Branch: master
Gerrit-Owner: Ejegg <eeggles...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to