ngsg commented on code in PR #6450:
URL: https://github.com/apache/hive/pull/6450#discussion_r3272608386


##########
itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/AbstractTestAuthorizationApiAuthorizer.java:
##########


Review Comment:
   I think the class name in the comment should also be updated.



##########
ql/src/test/queries/clientpositive/partitions_filter_default.q:
##########
@@ -1,3 +1,4 @@
+--! qt:disabled:HIVE-25965

Review Comment:
   I checked the Jenkins artifact and confirmed that the HIVE-25965 issue is 
being hit. However, I do not yet fully understand why this starts happening 
after applying this patch. Could you explain the expected connection between 
this patch and the failure?
   
   Also, I may have missed something, but I could not reproduce the issue in my 
local environment. Were you able to reproduce it locally?



-- 
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]

Reply via email to