rtrivedi12 commented on code in PR #4572:
URL: https://github.com/apache/hive/pull/4572#discussion_r1291327485
##########
ql/src/test/queries/clientpositive/auth_create_ext_table_1.q:
##########
@@ -0,0 +1,15 @@
+dfs ${system:test.dfs.mkdir} ${system:test.tmp.dir}/a_ext_create_tab1;
+dfs -touchz ${system:test.tmp.dir}/a_ext_create_tab1/1.txt;
+dfs -chmod 555 ${system:test.tmp.dir}/a_ext_create_tab1/1.txt;
+
+dfs ${system:test.dfs.mkdir} ${system:test.tmp.dir}/a_ext_create_tab2;
+dfs -chmod 555 ${system:test.tmp.dir}/a_ext_create_tab2;
+
+set
hive.metastore.pre.event.listeners=org.apache.hadoop.hive.ql.security.authorization.plugin.metastore.HiveMetaStoreAuthorizer;
+set
hive.security.authorization.manager=org.apache.hadoop.hive.ql.security.authorization.plugin.fallback.FallbackHiveAuthorizerFactory;
Review Comment:
@saihemanth-cloudera I agree with the thought of having a negative test for
CTAS. Since the exception is thrown at runtime while trying to create a staging
directory, the q.out file will not have any error/exception. Will it justify a
valid negative test scenario? I was not sure about this and excluded the test.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
