jenkins-bot has submitted this change and it was merged.
Change subject: Track request method in dbperformance.log
......................................................................
Track request method in dbperformance.log
This will allow us to avoid needlessly investigating master requests that
only happen on POST (which is already what we want for multi-DC).
Bug: T92357
Change-Id: Ia7437d00f5b89a8e318d85659d60e2f9f9f26149
---
M includes/profiler/TransactionProfiler.php
1 file changed, 3 insertions(+), 1 deletion(-)
Approvals:
BryanDavis: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/profiler/TransactionProfiler.php
b/includes/profiler/TransactionProfiler.php
index baec181..f02d66f 100644
--- a/includes/profiler/TransactionProfiler.php
+++ b/includes/profiler/TransactionProfiler.php
@@ -263,10 +263,12 @@
* @param string $query
*/
protected function reportExpectationViolated( $expect, $query ) {
+ global $wgRequest;
+
$n = $this->expect[$expect];
$by = $this->expectBy[$expect];
$this->logger->info(
- "Expectation ($expect <= $n) by $by not met:\n$query\n"
. wfBacktrace( true )
+ "[{$wgRequest->getMethod()}] Expectation ($expect <=
$n) by $by not met:\n$query\n" . wfBacktrace( true )
);
}
}
--
To view, visit https://gerrit.wikimedia.org/r/202008
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia7437d00f5b89a8e318d85659d60e2f9f9f26149
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Gilles <gdu...@wikimedia.org>
Gerrit-Reviewer: Aaron Schulz <asch...@wikimedia.org>
Gerrit-Reviewer: BryanDavis <bda...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
