JanusGraph / JanusGraph/janusgraph
Use ES6 High level Rest API when 6.0.0 will be released
- Dominant language
- Java
- Stars
- 5.8k
- Forks
- 1.2k
- Avg merge
- 13h 53m
- Merged PRs (30d)
- 6
Description
For ES6, we may use the new Java RestClient API https://www.elastic.co/guide/en/elasticsearch/client/java-rest/master/java-rest-high-compatibility.html ?
This api was released in 5.6.0 but "The 5.6 client can communicate with any 5.6.x Elasticsearch node. Previous 5.x minor versions like 5.5.x, 5.4.x etc. are not (fully) supported." So I think we should stay with the actual code for version < 6 but we should use this API for 6.
Contributor guide
Research direction
Start by reviewing JanusGraph's current Elasticsearch integration and the linked Java high-level REST client compatibility documentation. Confirm the 6.0.0 path can use the high-level API while older Elasticsearch versions retain the existing implementation, then verify compatibility for both version paths with the relevant integration tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- elasticsearch, java
- Domain
- backend, databases
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100