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-23786: HMS server side filter with Ranger #1221
Conversation
...org/apache/hadoop/hive/ql/metadata/TestSessionHiveMetastoreClientGetPartitionsTempTable.java
Outdated
Show resolved
Hide resolved
...org/apache/hadoop/hive/ql/metadata/TestSessionHiveMetastoreClientGetPartitionsTempTable.java
Outdated
Show resolved
Hide resolved
...oop/hive/ql/security/authorization/plugin/metastore/filtercontext/DatabaseFilterContext.java
Outdated
Show resolved
Hide resolved
...ache/hadoop/hive/ql/security/authorization/plugin/metastore/TestHiveMetaStoreAuthorizer.java
Outdated
Show resolved
Hide resolved
...etastore-server/src/test/java/org/apache/hadoop/hive/metastore/client/TestGetPartitions.java
Outdated
Show resolved
Hide resolved
...etastore-server/src/test/java/org/apache/hadoop/hive/metastore/client/TestGetPartitions.java
Outdated
Show resolved
Hide resolved
...g/apache/hadoop/hive/ql/security/authorization/plugin/metastore/HiveMetaStoreAuthorizer.java
Outdated
Show resolved
Hide resolved
...g/apache/hadoop/hive/ql/security/authorization/plugin/metastore/HiveMetaStoreAuthorizer.java
Outdated
Show resolved
Hide resolved
...g/apache/hadoop/hive/ql/security/authorization/plugin/metastore/HiveMetaStoreAuthorizer.java
Outdated
Show resolved
Hide resolved
...g/apache/hadoop/hive/ql/security/authorization/plugin/metastore/HiveMetaStoreAuthorizer.java
Show resolved
Hide resolved
...g/apache/hadoop/hive/ql/security/authorization/plugin/metastore/HiveMetaStoreAuthorizer.java
Outdated
Show resolved
Hide resolved
...g/apache/hadoop/hive/ql/security/authorization/plugin/metastore/HiveMetaStoreAuthorizer.java
Outdated
Show resolved
Hide resolved
...g/apache/hadoop/hive/ql/security/authorization/plugin/metastore/HiveMetaStoreAuthorizer.java
Outdated
Show resolved
Hide resolved
I have done a first sweeping review. A few asks:
As I am not too familiar with Ranger, please find a reviewer on that regard as well please. Thanks for the patch. We really need these checks on HMS side as well. |
One more thing: Do I remember correctly, that all the logs are on debug level? I think it would be good to have 1 or 2 log lines in info level, just t confirm that everything is good. Like "Filtered out 2 tables from 1000", or whatever... |
...org/apache/hadoop/hive/ql/security/authorization/plugin/metastore/events/ReadTableEvent.java
Outdated
Show resolved
Hide resolved
...org/apache/hadoop/hive/ql/security/authorization/plugin/metastore/events/ReadTableEvent.java
Outdated
Show resolved
Hide resolved
...etastore-server/src/test/java/org/apache/hadoop/hive/metastore/client/TestGetPartitions.java
Outdated
Show resolved
Hide resolved
...etastore-server/src/test/java/org/apache/hadoop/hive/metastore/client/TestGetPartitions.java
Outdated
Show resolved
Hide resolved
...g/apache/hadoop/hive/ql/security/authorization/plugin/metastore/HiveMetaStoreAuthorizer.java
Show resolved
Hide resolved
...g/apache/hadoop/hive/ql/security/authorization/plugin/metastore/HiveMetaStoreAuthorizer.java
Outdated
Show resolved
Hide resolved
...g/apache/hadoop/hive/ql/security/authorization/plugin/metastore/HiveMetaStoreAuthorizer.java
Outdated
Show resolved
Hide resolved
...hadoop/hive/ql/security/authorization/plugin/metastore/filtercontext/TableFilterContext.java
Outdated
Show resolved
Hide resolved
...hadoop/hive/ql/security/authorization/plugin/metastore/filtercontext/TableFilterContext.java
Outdated
Show resolved
Hide resolved
...etastore-server/src/test/java/org/apache/hadoop/hive/metastore/client/TestGetPartitions.java
Outdated
Show resolved
Hide resolved
6548f9f
to
15ec759
Compare
Fix has been merged into master. Thanks for the patch @sam-an-cloudera |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
HIVE-23786