jenkins-bot has submitted this change and it was merged.

Change subject: Hygiene: NoIndexExceptions are now expected
......................................................................


Hygiene: NoIndexExceptions are now expected

Initially all queries went through a cache, but we don't
want to maintain a cache for the various ways contributions
can be called.  Its usefull to have this message, but it's
currently just spamming up the production Flow.log when
contributions are hit.

Patch moves logs from Flow to FlowDebug group. Basically
proposing we split logs into Flow for prod and FlowDebug
for development.

Change-Id: Idf86d203d63b11ccbf539ff1cd3caad2e54cc3d2
---
M includes/Data/ObjectLocator.php
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Bsitu: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/Data/ObjectLocator.php b/includes/Data/ObjectLocator.php
index ce6974c..e956185 100644
--- a/includes/Data/ObjectLocator.php
+++ b/includes/Data/ObjectLocator.php
@@ -78,7 +78,7 @@
                        $index = $this->getIndexFor( $keys, $options );
                        $res = $index->findMulti( $queries, $options );
                } catch ( NoIndexException $e ) {
-                       wfDebugLog( 'Flow', __METHOD__ . ': ' . 
$e->getMessage() );
+                       wfDebugLog( 'FlowDebug', __METHOD__ . ': ' . 
$e->getMessage() );
                        $res = $this->storage->findMulti( $queries, 
$this->convertToDbOptions( $options ) );
                }
 
@@ -145,7 +145,7 @@
                        $res = $index->foundMulti( $queries, $options );
                        return $res;
                } catch ( NoIndexException $e ) {
-                       wfDebugLog( 'Flow', __METHOD__ . ': ' . 
$e->getMessage() );
+                       wfDebugLog( 'FlowDebug', __METHOD__ . ': ' . 
$e->getMessage() );
                }
 
                return false;

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Idf86d203d63b11ccbf539ff1cd3caad2e54cc3d2
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: EBernhardson <[email protected]>
Gerrit-Reviewer: Bsitu <[email protected]>
Gerrit-Reviewer: SG <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to