ppkarwasz commented on PR #3901: URL: https://github.com/apache/logging-log4j2/pull/3901#issuecomment-3239234235
Hi @ramanathan1504, thanks for the PR! After checking the [Elasticsearch Java client 9.0.0 release notes](https://www.elastic.co/docs/release-notes/elasticsearch/clients/java/9-0-0): * The Java baseline is now **JRE 17**. Our CI runs tests on **JRE 8**, so the integration test will fail there even if it passes locally (you can reproduce locally by setting `CI=true`). * The legacy **`RestClient`** was deprecated in favor of **`Rest5Client`** (based on Apache HttpComponents 5.x, since version 4.x is EOL). **Proposal** 1. Upgrade `main` to use `Rest5Client`. 2. Keep the `2.x` branch on Elasticsearch **8.x** and update `.github/dependabot.yaml` to ignore `9.x` (and above) proposals. 3. Add a reminder to drop 8.x when it goes EOL in \~2 years (see the [EOL policy](https://www.elastic.co/support/eol#prior-versions)) by annotating the test class with JUnit Pioneer: ```java @FailAt( date = "2027-07-15", reason = "Elasticsearch 8.x is no longer supported" ) ``` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
