Skip to content

Deprecate client.type#104574

Merged
elasticsearchmachine merged 5 commits intoelastic:mainfrom
rjernst:clienttype/deprecate
Feb 6, 2024
Merged

Deprecate client.type#104574
elasticsearchmachine merged 5 commits intoelastic:mainfrom
rjernst:clienttype/deprecate

Conversation

@rjernst
Copy link
Member

@rjernst rjernst commented Jan 19, 2024

client.type existed from the days of the node client existing alongside the java client. Since the node client no longer exists, it no longer serves a purpose, and is already ignored. Yet the setting still exists. This commit deprecates the client.type node setting.

client.type existed from the days of the node client existing alongside
the java client. Since the node client no longer exists, it no longer
serves a purpose, and is already ignored. Yet the setting still exists.
This commit deprecates the client.type node setting.
@rjernst rjernst added :Core/Infra/Core Core issues without another label >deprecation v8.13.0 labels Jan 19, 2024
@rjernst rjernst requested a review from a team January 19, 2024 00:28
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Jan 19, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@elasticsearchmachine
Copy link
Collaborator

Hi @rjernst, I've created a changelog YAML for you. Note that since this PR is labelled >deprecation, you need to update the changelog YAML to fill out the extended information sections.

Copy link
Contributor

@ldematte ldematte left a comment

Choose a reason for hiding this comment

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

lgtm

@rjernst rjernst added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Feb 6, 2024
@elasticsearchmachine elasticsearchmachine merged commit b1e2cb8 into elastic:main Feb 6, 2024
@rjernst rjernst deleted the clienttype/deprecate branch February 6, 2024 04:16
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Dec 6, 2024
The client.type setting is a holdover from the node client which was
removed in 8.0. The setting has been a noop since 8.0. This commit
removes the setting.

relates elastic#104574
rjernst added a commit that referenced this pull request Dec 9, 2024
The client.type setting is a holdover from the node client which was
removed in 8.0. The setting has been a noop since 8.0. This commit
removes the setting.

relates #104574
@leemthompo
Copy link
Contributor

@rjernst is this PR relevant to the serverless changelog? [FYI this question is based on 9.0 breaking changes]

@rjernst
Copy link
Member Author

rjernst commented Apr 11, 2025

No, this does not affect serverless at all.

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

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Core/Infra/Core Core issues without another label >deprecation Team:Core/Infra Meta label for core/infra team v8.13.0

4 participants