-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
HBASE-22613: Bump Apache Avro to 1.9.0 #326
Conversation
🎊 +1 overall
This message was automatically generated. |
It seems that we do not depend on avro directly in HBase... |
As I mentioned on jira, we have avro listed in order to harmonize getting it from multiple places as a transitive dependency. Are we no longer getting it from multiple places? |
At least we do not have an explicit dependency of avro now in HBase. Maybe we should add one to force the version? |
💔 -1 overall
This message was automatically generated. |
The process of harmonizing Avro is ongoing: HADOOP-13386: Bump Apache Avro to 1.9.0 It is going quite smoothly. If hbase doesn't have a direct dependency on Avro, I would argue that it isn't up to hbase to set the version then. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
Close since no progress here. And I think the main problem here is whether we need the avro dependency. Feel free to reopen after we have a consensus. |
I've compiled it locally and expected some breaking changes since we're upgrading from 1.7.7. But I did not find anything.
Apache Avro 1.9.0 brings a lot of new features: