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

Change subject: timestamp -> ts in CirrusSearchRequestSet
......................................................................


timestamp -> ts in CirrusSearchRequestSet

Prevent potential issues with utilizing this schema in other utilities
where 'timestamp' might be a keyword.

Change-Id: I576c61cdbea5e6c0de6c23a7f7e9dc22dc9cd517
---
M includes/ElasticsearchIntermediary.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Cindy-the-browser-test-bot: Looks good to me, but someone else must approve
  DCausse: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/ElasticsearchIntermediary.php 
b/includes/ElasticsearchIntermediary.php
index cca9ed3..7645014 100644
--- a/includes/ElasticsearchIntermediary.php
+++ b/includes/ElasticsearchIntermediary.php
@@ -173,7 +173,7 @@
                }
 
                $requestSet = array(
-                       'timestamp' => time(),
+                       'ts' => time(),
                        'wikiId' => wfWikiId(),
                        'source' => self::getExecutionContext(),
                        'identity' => self::generateIdentToken(),

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I576c61cdbea5e6c0de6c23a7f7e9dc22dc9cd517
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/CirrusSearch
Gerrit-Branch: master
Gerrit-Owner: EBernhardson <[email protected]>
Gerrit-Reviewer: Chad <[email protected]>
Gerrit-Reviewer: Cindy-the-browser-test-bot <[email protected]>
Gerrit-Reviewer: DCausse <[email protected]>
Gerrit-Reviewer: Manybubbles <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to