Skip to content

Bump org.elasticsearch.client:elasticsearch-rest-client from 9.3.4 to 9.4.0 in /elasticsearch#113

Open
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/maven/elasticsearch/org.elasticsearch.client-elasticsearch-rest-client-9.4.0
Open

Bump org.elasticsearch.client:elasticsearch-rest-client from 9.3.4 to 9.4.0 in /elasticsearch#113
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/maven/elasticsearch/org.elasticsearch.client-elasticsearch-rest-client-9.4.0

Conversation

@dependabot
Copy link
Copy Markdown
Contributor

@dependabot dependabot Bot commented on behalf of github May 27, 2026

Bumps org.elasticsearch.client:elasticsearch-rest-client from 9.3.4 to 9.4.0.

Release notes

Sourced from org.elasticsearch.client:elasticsearch-rest-client's releases.

Elasticsearch 9.4.0

Downloads: https://elastic.co/downloads/elasticsearch Release notes: https://www.elastic.co/docs/release-notes/elasticsearch#elasticsearch-9.4.0-release-notes

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [org.elasticsearch.client:elasticsearch-rest-client](https://github.com/elastic/elasticsearch) from 9.3.4 to 9.4.0.
- [Release notes](https://github.com/elastic/elasticsearch/releases)
- [Changelog](https://github.com/elastic/elasticsearch/blob/main/docs/changelog.yml)
- [Commits](elastic/elasticsearch@v9.3.4...v9.4.0)

---
updated-dependencies:
- dependency-name: org.elasticsearch.client:elasticsearch-rest-client
  dependency-version: 9.4.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot requested a review from a team as a code owner May 27, 2026 03:20
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file java Pull requests that update Java code labels May 27, 2026
@github-actions
Copy link
Copy Markdown

🤖 Claude Code Review


Code Review

PR: Bump org.elasticsearch.client:elasticsearch-rest-client from 9.3.49.4.0


Code Quality

✅ No commented-out code — change is a single version string update.

✅ Meaningful variable names — N/A for a version bump.

✅ DRY principle — N/A.

❌ Potential defect — version mismatch between related dependencies
At elasticsearch/pom.xml line 73, co.elastic.clients:elasticsearch-java remains pinned at 9.3.4, while org.elasticsearch.client:elasticsearch-rest-client is now 9.4.0 (line 78). These two artifacts are part of the same Elastic release train and must be kept at the same version. A minor version mismatch (9.3.x client against a 9.4.x low-level REST client) can cause subtle runtime failures or API incompatibilities. Both should be bumped together to 9.4.0.

❌ Inconsistent indentation in pom.xml
The pom.xml mixes tabs and spaces across dependency blocks (e.g., lines 65–109 vs. lines 85–89). This is a pre-existing issue but worth noting; the new dependency block (lines 75–79) uses a different indent style than its neighbours.


Testing

❌ No tests added or updated — there are no visible test changes. While a dependency bump may not require new unit tests, a smoke test or integration test confirming the new REST client version works would be appropriate.


Documentation

❌ CHANGELOG.md not updatedCHANGELOG.md has no entry for this dependency bump. Per the project's own format (Keep a Changelog), dependency upgrades should be documented.


Security

✅ No hardcoded credentials.

✅ No sensitive data or license files.

✅ No .lic files or AQAAAD-prefixed content.


Summary

Severity Finding
High co.elastic.clients:elasticsearch-java (line 73) is still 9.3.4 — must be bumped to 9.4.0 to match the REST client
Low CHANGELOG.md not updated
Low pom.xml mixed indentation (pre-existing, but worsened by the new block)

The version mismatch between the two Elastic artifacts is the critical issue to resolve before merging.

Automated code review analyzing defects and coding standards

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

Labels

dependencies Pull requests that update a dependency file java Pull requests that update Java code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants