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

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

                Author: ASF GitHub Bot
            Created on: 12/May/22 13:05
            Start Date: 12/May/22 13:05
    Worklog Time Spent: 10m 
      Work Description: ayushtkn commented on code in PR #3279:
URL: https://github.com/apache/hive/pull/3279#discussion_r871355730


##########
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java:
##########
@@ -9361,7 +9362,8 @@ public NotificationEventsCountResponse 
get_notification_events_count(Notificatio
   private void authorizeProxyPrivilege() throws TException {
     // Skip the auth in embedded mode or if the auth is disabled
     if (!HiveMetaStore.isMetaStoreRemote() ||
-        !MetastoreConf.getBoolVar(conf, 
ConfVars.EVENT_DB_NOTIFICATION_API_AUTH)) {
+        !MetastoreConf.getBoolVar(conf, 
ConfVars.EVENT_DB_NOTIFICATION_API_AUTH) || 
conf.getBoolean(HIVE_IN_TEST.getVarname(),
+        false)) {

Review Comment:
   It is covered via test in 
TestReplicationScenarios#testAuthForNotificationAPIs
   This method is also used mostly in replication context only I suppose for 
getting NotificationLog entries...





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

    Worklog Id:     (was: 769620)
    Time Spent: 9.05h  (was: 8h 53m)

> 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: 9.05h
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to