Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove Deprecated TransportClient #1669

Closed
saratvemulapalli opened this issue Dec 7, 2021 · 0 comments · Fixed by #1781
Closed

Remove Deprecated TransportClient #1669

saratvemulapalli opened this issue Dec 7, 2021 · 0 comments · Fixed by #1781
Assignees
@adnapibar
Labels
>breaking Identifies a breaking change. enhancement Enhancement or improvement to existing feature or request non-issue bugs / unexpected behaviors that end up non issues; audit trail simple changes that aren't issues Priority-Low v2.0.0 Version 2.0.0

Comments

@saratvemulapalli
Copy link
Member

saratvemulapalli commented Dec 7, 2021 •
edited
Loading

Is your feature request related to a problem? Please describe.
TransportClient.java is deprecated and should be removed and adopt using High Level Rest Client.

https://github.com/opensearch-project/OpenSearch/blob/main/server/src/main/java/org/opensearch/client/transport/TransportClient.java

There are a lot of references where TransportClient is used, all of the uses cases should be moved a layer above of TCP to Rest (Layer 7).

The intent of this issue is to sanitizing the code base.
@saratvemulapalli saratvemulapalli added enhancement Enhancement or improvement to existing feature or request untriaged Priority-Low labels Dec 7, 2021
@saratvemulapalli saratvemulapalli mentioned this issue Dec 8, 2021
Closed
19 tasks
@nknize nknize added the non-issue bugs / unexpected behaviors that end up non issues; audit trail simple changes that aren't issues label Dec 8, 2021
@nknize nknize changed the title TransportClient is deprecated but not removed yet Remove Deprecated TransportClient Dec 8, 2021
@adnapibar adnapibar self-assigned this Dec 9, 2021
This was referenced Dec 20, 2021
Merged
Merged
@davidlago davidlago mentioned this issue Jan 19, 2022
Closed
@saratvemulapalli saratvemulapalli added the >breaking Identifies a breaking change. label Feb 15, 2022
@rs-eliatra rs-eliatra mentioned this issue Feb 23, 2022
Merged
@saratvemulapalli saratvemulapalli added the v2.0.0 Version 2.0.0 label Mar 2, 2022
@saratvemulapalli saratvemulapalli mentioned this issue Mar 15, 2022
Closed
10 tasks
@peternied peternied mentioned this issue Aug 18, 2022
Closed
4 tasks
@davidlago davidlago mentioned this issue Oct 25, 2022
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adnapibar adnapibar

Labels
>breaking Identifies a breaking change. enhancement Enhancement or improvement to existing feature or request non-issue bugs / unexpected behaviors that end up non issues; audit trail simple changes that aren't issues Priority-Low v2.0.0 Version 2.0.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove deprecated transport client
4 participants
@nknize @adnapibar @saratvemulapalli @anasalkouz