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

Deprecated the thrift and memcached transports #10167

Merged
merged 1 commit into from Mar 19, 2015
Merged

Deprecated the thrift and memcached transports #10167

merged 1 commit into from Mar 19, 2015

Conversation

clintongormley
Copy link

The thrift and memcached transport layers were experimental, but in
the end didn't buy us much. Memcached is very limited in scope,
supporting only a portion of the REST API, and thrift has much
the same throughput as HTTP.

They are deprecated as of 1.5.0, and will be removed in 2.0.

Document changes for #10166

The thrift and memcached transport layers were experimental, but in
the end didn't buy us much. Memcached is very limited in scope,
supporting only a portion of the REST API, and thrift has much
the same throughput as HTTP.

They are deprecated as of 1.5.0, and will be removed in 2.0.

Document changes for #10166
@clintongormley clintongormley added v1.5.0 :Distributed/Network Http and internode communication implementations v1.6.0 >deprecation labels Mar 19, 2015
clintongormley added a commit that referenced this pull request Mar 19, 2015
Deprecated the thrift and memcached transports
@clintongormley clintongormley merged commit 7f69016 into elastic:1.5 Mar 19, 2015
@im-denisenko im-denisenko mentioned this pull request Apr 6, 2015
16 tasks
@ruflin
Copy link
Member

ruflin commented May 20, 2015

@clintongormley I assume Thrift will also be removed in version 2.0? I'm asking as you only marked as deprecated in the memcache file and not in the Thrift file: https://github.com/elastic/elasticsearch/blob/d63bd7228beb79a957d9b7bef059487f3b9f7d81/docs/reference/modules/thrift.asciidoc

@clintongormley
Copy link
Author

@ruflin thanks for spotting - i've added the missing deprecation notice dbcc08f

@ruflin
Copy link
Member

ruflin commented May 25, 2015

👍

mute pushed a commit to mute/elasticsearch that referenced this pull request Jul 29, 2015
…_memcached

Deprecated the thrift and memcached transports
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>deprecation :Distributed/Network Http and internode communication implementations v1.5.0 v1.6.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants