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)
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
Unknown Object (File)
Mar 15 2017, 3:38 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

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