Page MenuHomePhabricator

Fix setup warning on elasticsearch 5.0
ClosedPublic

Authored by mmodell on Feb 10 2017, 9:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 7, 4:49 AM
Unknown Object (File)
Apr 12 2017, 10:06 AM
Unknown Object (File)
Apr 12 2017, 10:05 AM
Unknown Object (File)
Apr 12 2017, 10:05 AM
Unknown Object (File)
Apr 5 2017, 9:49 PM
Unknown Object (File)
Mar 30 2017, 1:33 PM
Unknown Object (File)
Mar 29 2017, 1:46 AM
Unknown Object (File)
Mar 18 2017, 8:54 AM
Subscribers

Details

Maniphest Tasks
T155299: Phabricator: Support elasticsearch 5.x
Reviewers
Paladox
MZMcBride
EBernhardson
Patch without arc
git checkout -b D563 && curl -L https://phabricator.wikimedia.org/D563?download=true | git apply
Summary

This Improves ElasticSearch 5.0 compatibility and fixes the
setup check to properly detect missing index & outdated mappings
without false positive on ElasticSearch 5+

Test Plan

Tested with es 5.2:

  • bin/search init
  • bin/search index
  • execute some test queries in phabricator

Not tested:

  • es 2.x support

Diff Detail

Repository
rPHAB Phabricator
Branch
es5 (branched from wmf/stable)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1603
Build 2562: differential-jessieJenkins
Build 2561: arc lint + arc unit

Event Timeline

Yay that fixed it, thanks :)

This revision is now accepted and ready to land.Feb 10 2017, 9:49 PM

Deployed on phab-01 and it works :)

use old style php array syntax to make lint happy