[ 
https://issues.apache.org/jira/browse/HIVE-24484?focusedWorklogId=626721&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-626721
 ]

ASF GitHub Bot logged work on HIVE-24484:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 22/Jul/21 14:47
            Start Date: 22/Jul/21 14:47
    Worklog Time Spent: 10m 
      Work Description: abstractdog commented on a change in pull request #1742:
URL: https://github.com/apache/hive/pull/1742#discussion_r674871945



##########
File path: 
itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/parse/TestReplicationScenarios.java
##########
@@ -4117,28 +4118,33 @@ public void testAuthForNotificationAPIs() throws 
Exception {
     createDB(dbName, driver);
     NotificationEventResponse rsp = 
metaStoreClient.getNextNotification(firstEventId, 0, null);
     assertEquals(1, rsp.getEventsSize());
+
     // Test various scenarios
-    // Remove the proxy privilege and the auth should fail (in reality the 
proxy setting should not be changed on the fly)
-    hconf.unset(proxySettingName);
-    // Need to explicitly update ProxyUsers
-    ProxyUsers.refreshSuperUserGroupsConfiguration(hconf);
-    // Verify if the auth should fail
-    Exception ex = null;
+    // Remove the proxy privilege by reseting proxy configuration to default 
value.
+    // The auth should fail (in reality the proxy setting should not be 
changed on the fly)
+    // Pretty hacky: Affects both instances of HMS
+    ProxyUsers.refreshSuperUserGroupsConfiguration();
+
     try {
       rsp = metaStoreClient.getNextNotification(firstEventId, 0, null);
+      Assert.fail("Get Next Nofitication should have failed due to no proxy 
auth");
     } catch (TException e) {
-      ex = e;

Review comment:
       I have no idea how can we hit this catch, but having it empty is always 
a red sign, have you checked if at least a log.debug is useful here?




-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
-------------------

    Worklog Id:     (was: 626721)
    Time Spent: 3h 23m  (was: 3h 13m)

> Upgrade Hadoop to 3.3.1
> -----------------------
>
>                 Key: HIVE-24484
>                 URL: https://issues.apache.org/jira/browse/HIVE-24484
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: David Mollitor
>            Assignee: David Mollitor
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 3h 23m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to