New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move to Elasticsearch RESTAPi #539

Merged
merged 4 commits into from Mar 14, 2018

Conversation

Projects
None yet
1 participant
@jnioche
Member

jnioche commented Mar 5, 2018

The TransportClient won't be supported in future versions, currently does not work with Elastic Cloud and is difficult to configure when the access to ES is secured.
This PR uses the REST high-level client. The main differences with the master branch are:

  • addresses must be fully formed URLs e.g. http://localhost:9200
  • removed util/URLExtractor.java (could be ported in a separate PR if necessary)
  • Spouts don't check the number of shards and just assign them to instances simply based on the instance number
  • secure access to ES using
es.status.user: "USERNAME"
es.status.password: "PASSWORD"

Reviews and tests would be greatly appreciated

@jnioche jnioche added this to the 1.8 milestone Mar 5, 2018

@jnioche jnioche self-assigned this Mar 5, 2018

@jnioche jnioche requested a review from jorgelbg Mar 5, 2018

@jnioche jnioche merged commit f6d51de into master Mar 14, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@jnioche jnioche deleted the ESRestAPi branch Mar 14, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment