Skip to content

Normalize Elasticsearch API client javaagent package#18730

Merged
trask merged 1 commit into
open-telemetry:mainfrom
trask:package-name-cleanup-elasticsearch-api-client
May 14, 2026
Merged

Normalize Elasticsearch API client javaagent package#18730
trask merged 1 commit into
open-telemetry:mainfrom
trask:package-name-cleanup-elasticsearch-api-client

Conversation

@trask

@trask trask commented May 13, 2026

Copy link
Copy Markdown
Member

@trask trask marked this pull request as ready for review May 13, 2026 13:59
@trask trask requested a review from a team as a code owner May 13, 2026 13:59
Copilot AI review requested due to automatic review settings May 13, 2026 13:59

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR normalizes the package name for the Elasticsearch API client (7.16) javaagent instrumentation to match its directory structure and align with the broader package/module naming alignment effort referenced in #18428.

Changes:

  • Renamed javaagent main-source packages from ...elasticsearch.apiclient to ...elasticsearch.api.client.v7_16.
  • Updated impacted static imports to the new package path.
  • Updated unit-test imports/packages to reference the new package.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
instrumentation/elasticsearch/elasticsearch-api-client-7.16/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/elasticsearch/api/client/v7_16/ElasticsearchClientTest.java Updates test package declaration to the normalized ...api.client.v7_16 namespace.
instrumentation/elasticsearch/elasticsearch-api-client-7.16/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/elasticsearch/api/client/v7_16/RestClientTransportInstrumentation.java Updates package declaration and static import to match the new namespace.
instrumentation/elasticsearch/elasticsearch-api-client-7.16/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/elasticsearch/api/client/v7_16/RestClientHttpClientInstrumentation.java Updates package declaration and static import to match the new namespace.
instrumentation/elasticsearch/elasticsearch-api-client-7.16/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/elasticsearch/api/client/v7_16/EndpointId.java Updates package declaration to the normalized namespace.
instrumentation/elasticsearch/elasticsearch-api-client-7.16/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/elasticsearch/api/client/v7_16/ElasticsearchEndpointMap.java Updates package declaration to the normalized namespace.
instrumentation/elasticsearch/elasticsearch-api-client-7.16/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/elasticsearch/api/client/v7_16/ElasticsearchApiClientSingletons.java Updates package declaration to the normalized namespace.
instrumentation/elasticsearch/elasticsearch-api-client-7.16/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/elasticsearch/api/client/v7_16/ElasticsearchApiClientInstrumentationModule.java Updates package declaration to the normalized namespace.
instrumentation/elasticsearch/elasticsearch-api-client-7.16/javaagent-unit-tests/src/test/java/io/opentelemetry/javaagent/instrumentation/elasticsearch/rest/ElasticsearchEndpointMapTest.java Updates import to reference ElasticsearchEndpointMap in the new package.

@trask trask merged commit e9c3259 into open-telemetry:main May 14, 2026
99 checks passed
@trask trask deleted the package-name-cleanup-elasticsearch-api-client branch May 14, 2026 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants