jenkins-bot has submitted this change and it was merged. Change subject: Update Elasticsearch Searcher for logging refactor ......................................................................
Update Elasticsearch Searcher for logging refactor Change-Id: I367f28a9d74c2a206e73ab58cbcd141cec8a2272 Depends-On: I50761d7dfb32231cd75c1f93e7a804f177d80b24 --- M includes/Searcher.php 1 file changed, 9 insertions(+), 2 deletions(-) Approvals: DCausse: Looks good to me, approved jenkins-bot: Verified diff --git a/includes/Searcher.php b/includes/Searcher.php index 03c6cbb..02d5594 100644 --- a/includes/Searcher.php +++ b/includes/Searcher.php @@ -5,6 +5,7 @@ use CirrusSearch\ElasticsearchIntermediary; use CirrusSearch\SearchConfig; +use CirrusSearch\SearchRequestLog; use ConfigFactory; use Elastica\Exception\ExceptionInterface; use User; @@ -35,8 +36,14 @@ $pageType = $this->connection->getPageType( wfWikiID(), $indexType ); $search = $pageType->createSearch( $query ); + $log = new SearchRequestLog( + $this->connection->getClient(), + $this->user, + 'performing {queryType}', + $queryType + ); try { - $this->start( "performing $queryType", [ 'queryType' => $queryType ] ); + $this->start( $log ); $result = $search->search(); $this->success(); } catch ( ExceptionInterface $ex ) { @@ -46,4 +53,4 @@ return $result; } -} \ No newline at end of file +} -- To view, visit https://gerrit.wikimedia.org/r/314350 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I367f28a9d74c2a206e73ab58cbcd141cec8a2272 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/GeoData Gerrit-Branch: master Gerrit-Owner: EBernhardson <ebernhard...@wikimedia.org> Gerrit-Reviewer: DCausse <dcau...@wikimedia.org> Gerrit-Reviewer: MaxSem <maxsem.w...@gmail.com> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits