Replace pelias-esclient with official elasticsearch module #151

Merged
merged 1 commit into from Jul 28, 2016

Conversation

Projects
None yet
3 participants
@orangejulius
Member

orangejulius commented Jul 28, 2016

Now that the client is deprecated we should stop using it. Fortunately, esclient was mostly a wrapper around the official Elasticsearch client, so this was a straightforward change.

Fixes #144

Replace pelias-esclient with official elasticsearch module
Now that the client is [deprecated](https://github.com/pelias-deprecated/esclient) we should
stop using it.

Fixes #144

@orangejulius orangejulius self-assigned this Jul 28, 2016

@trescube

This comment has been minimized.

Show comment
Hide comment
@trescube

trescube Jul 28, 2016

Contributor

:shipit:

Contributor

trescube commented Jul 28, 2016

:shipit:

@dianashk

This comment has been minimized.

Show comment
Hide comment
@dianashk

dianashk Jul 28, 2016

Contributor

LGTM

Contributor

dianashk commented Jul 28, 2016

LGTM

@orangejulius orangejulius merged commit afe3ec1 into master Jul 28, 2016

3 checks passed

approvals/lgtm this commit looks good
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@orangejulius orangejulius removed the in review label Jul 28, 2016

@orangejulius orangejulius deleted the elasticsearch-client branch Jul 28, 2016

je-l pushed a commit to nlsfi/pelias-schema that referenced this pull request Aug 31, 2017

Merge pull request #151 from pelias/elasticsearch-client
Replace pelias-esclient with official elasticsearch module
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment