Fix setup warning on elasticsearch 5.0
ClosedPublic

Authored by mmodell on Feb 10 2017, 9:43 PM.

Details

Maniphest Tasks
T155299: Phabricator: Support elasticsearch 5.x
Reviewers
Paladox
MZMcBride
EBernhardson
Commits
rPHABd2f8fe6e4e79: Fix setup warning on elasticsearch 5.0
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 OK
Unit
Unit Tests OK
Build Status
Buildable 1603
Build 2562: differential-jessieJenkins
Build 2561: arc lint + arc unit
mmodell created this revision.Feb 10 2017, 9:43 PM
Paladox accepted this revision.Feb 10 2017, 9:49 PM

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 :)

mmodell updated this revision to Diff 1491.Feb 10 2017, 9:51 PM

use old style php array syntax to make lint happy

mmodell closed this revision.Feb 10 2017, 9:53 PM