-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
HIVE-26537: Deprecate older APIs in the HMS thrift interface. #3599
Conversation
standalone-metastore/metastore-common/src/main/thrift/hive_metastore.thrift
Show resolved
Hide resolved
standalone-metastore/metastore-common/src/main/thrift/hive_metastore.thrift
Outdated
Show resolved
Hide resolved
...ne-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java
Outdated
Show resolved
Hide resolved
...ne-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java
Show resolved
Hide resolved
3ec9918
to
31e03d0
Compare
standalone-metastore/metastore-common/src/main/thrift/hive_metastore.thrift
Show resolved
Hide resolved
...ne-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java
Outdated
Show resolved
Hide resolved
...ne-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java
Show resolved
Hide resolved
The changes in |
Quality Gate passedIssues Measures |
+1, cc @nrg4878 @deniskuzZ @ayushtkn it would be great if another eye will take here before it gets merged. Thank you in advance! |
…3599) (Sai Hemanth Gantasala, reviewed by Zhihua Deng, Naveen Gangam)
What changes were proposed in this pull request?
Deprecated older APIs in HMS and pointed them to the newer APIs
Why are the changes needed?
Going forward HMS should be using the deprecated APIs.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Unit tests.