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.

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

Deprecate the transport client in favour of the high-level REST client #27085

Merged
merged 3 commits into from Dec 1, 2017

Conversation

javanna
Copy link
Member

@javanna javanna commented Oct 23, 2017

This PR deprecates the transport client in our docs and in the java code.

@javanna
Copy link
Member Author

javanna commented Oct 23, 2017

I targeted this for 6.0.0 too although it is probably too late, in which case we can simply push it to 6.1.0 .

@xeraa
Copy link
Contributor

xeraa commented Oct 26, 2017

Adding another data point here: I tweeted about this PR (probably a bit prematurely, I'm sorry for that). While this is not representative and reactions are hard to judge, only one person actively complained.

lhtx3cfze6 time to jump o 2017-10-26 15-29-27

@javanna
Copy link
Member Author

javanna commented Nov 3, 2017

We discussed this in FixItFriday and decided to wait. The idea is to work on adding support for the missing API to the high-level REST client, deprecate transport client in 7.0 and remove it in 8.0.

@javanna
Copy link
Member Author

javanna commented Nov 6, 2017

I brought this PR up-to-date. @clintongormley do we want to get this in master now or wait till more API are supported by the high-level REST client?

@clintongormley
Copy link

Let's merge this into master only. Will act as a forcing function :)

@javanna javanna force-pushed the deprecation/transport_client branch from c1e6996 to 1dd0c67 Compare December 1, 2017 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants