----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67102/#review203070 -----------------------------------------------------------
service/src/java/org/apache/hive/service/server/HiveServer2.java Lines 992 (patched) <https://reviews.apache.org/r/67102/#comment285140> how about using AuthorizationPreEventListener.class.getName() or so ? - Thejas Nair On May 15, 2018, 9:04 p.m., Daniel Dai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67102/ > ----------------------------------------------------------- > > (Updated May 15, 2018, 9:04 p.m.) > > > Review request for hive. > > > Repository: hive-git > > > Description > ------- > > See HIVE-19440 > > > Diffs > ----- > > > hcatalog/core/src/main/java/org/apache/hive/hcatalog/storagehandler/DummyHCatAuthProvider.java > a53028f > > itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/security/TestHDFSPermissionPolicyProvider.java > PRE-CREATION > > itests/util/src/main/java/org/apache/hadoop/hive/ql/security/DummyHiveMetastoreAuthorizationProvider.java > 31e795c > metastore/scripts/upgrade/hive/hive-schema-3.0.0.hive.sql d9606d8 > ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java a1f549a > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/HDFSPermissionPolicyProvider.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/HiveAuthorizationProviderBase.java > 8a7c06d > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/HiveMetastoreAuthorizationProvider.java > 0dab334 > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/PolicyProviderContainer.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/PrivilegeSynchonizer.java > 9b2e6cd > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/StorageBasedAuthorizationProvider.java > b66d188 > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveV1Authorizer.java > 48798d8 > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLAuthorizationUtils.java > 02ed7aa > > ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFCurrentAuthorizer.java > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFRestrictInformationSchema.java > 3eb0914 > service/src/java/org/apache/hive/service/server/HiveServer2.java 661beb5 > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java > 92d2e3f > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java > 6af2aa5 > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java > 09f9bb1 > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java > 264fdb9 > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/RawStore.java > ce7d286 > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/cache/CachedStore.java > b223920 > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/client/builder/HiveObjectPrivilegeBuilder.java > d802e1a > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/model/MDBPrivilege.java > 3d8fa21 > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/model/MGlobalPrivilege.java > 5b496e0 > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/model/MPartitionColumnPrivilege.java > ab50a92 > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/model/MPartitionPrivilege.java > 3193bc1 > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/model/MTableColumnPrivilege.java > ad7322f > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/model/MTablePrivilege.java > 6460400 > standalone-metastore/src/main/resources/package.jdo 2d2cb19 > standalone-metastore/src/main/sql/derby/hive-schema-3.1.0.derby.sql > PRE-CREATION > standalone-metastore/src/main/sql/derby/upgrade-3.0.0-to-3.1.0.derby.sql > PRE-CREATION > standalone-metastore/src/main/sql/mssql/hive-schema-3.1.0.mssql.sql > PRE-CREATION > standalone-metastore/src/main/sql/mssql/upgrade-3.0.0-to-3.1.0.mssql.sql > PRE-CREATION > standalone-metastore/src/main/sql/mysql/hive-schema-3.1.0.mysql.sql > PRE-CREATION > standalone-metastore/src/main/sql/mysql/upgrade-3.0.0-to-3.1.0.mysql.sql > PRE-CREATION > standalone-metastore/src/main/sql/oracle/hive-schema-3.1.0.oracle.sql > PRE-CREATION > standalone-metastore/src/main/sql/oracle/upgrade-3.0.0-to-3.1.0.oracle.sql > PRE-CREATION > standalone-metastore/src/main/sql/postgres/hive-schema-3.1.0.postgres.sql > PRE-CREATION > > standalone-metastore/src/main/sql/postgres/upgrade-3.0.0-to-3.1.0.postgres.sql > PRE-CREATION > standalone-metastore/src/main/thrift/hive_metastore.thrift 19d4433 > > standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/DummyRawStoreControlledCommit.java > f6899be > > standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/DummyRawStoreForJdoConnection.java > 98a85cc > > standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClientPreCatalog.java > 7186add > > > Diff: https://reviews.apache.org/r/67102/diff/2/ > > > Testing > ------- > > > Thanks, > > Daniel Dai > >
