-
Notifications
You must be signed in to change notification settings - Fork 1
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
Support Thrift2 API #4
Comments
Getting that StackOverflow warm welcome I see, haha. In brief, the thrift2 API was written to more closely match the Java API it is wrapping. Both APIs are still fully supported, but some day the original thrift API may be deprecated (at least that's what the docs say). However, I very much doubt that would happen as both APIs have been around for a very long time and the original one still seems be used more. It does indeed appear that batch increments are not supported. I don't know the fine internal details of why that feature would be removed, but if that is one of your primary use cases, it might be best to see if you can also enable the original thrift API and use that. There's no reason you couldn't have both running on two different ports. |
Thank you for the answer! |
Let me know if you need anything else. I don't use the thrift2 API at work, so this isn't the highest priority for me but I can raise it up if you definitely need it. |
I'm trying thrift (1) now, I'll have to verify the performance and I'll update later on, thanks a lot @aiudirog ! |
Splitting this out from #3 as a dedicated issue to support the HBase thrift2 API.
The text was updated successfully, but these errors were encountered: