[GitHub] [jackrabbit-oak] nfsantos commented on a diff in pull request #897: OAK-10187: oak-search-elastic: update Java Client independently of HLRC

["nfsantos (via GitHub)" <gi...@apache.org>]

nfsantos commented on code in PR #897:
URL: https://github.com/apache/jackrabbit-oak/pull/897#discussion_r1166358140


##########
oak-search-elastic/pom.xml:
##########
@@ -33,7 +33,8 @@
   <description>Oak Elasticsearch integration subproject</description>
 
   <properties>
-    <elasticsearch.version>7.17.7</elasticsearch.version>
+    <elasticsearch.hlrc.version>7.17.9</elasticsearch.hlrc.version>
+    <elasticsearch.java.version>8.5.3</elasticsearch.java.version>

Review Comment:
   Any reason for not using the latest version at this time, 8.7.0? It should be compatible with any 8.X server. 
   And maybe more importantly, 8.7.0 has a bulk ingester helper that at a first glance, seems to be what we need to drop the 7.17.X client. 
   https://github.com/elastic/elasticsearch-java/pull/474
   https://github.com/elastic/elasticsearch-java/releases/tag/v8.7.0



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@jackrabbit.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org