dengzhhu653 commented on code in PR #6829:
URL: https://github.com/apache/hive/pull/6829#discussion_r4332241336
##########
standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/client/MetaStoreClientTest.java:
##########
@@ -74,9 +74,14 @@ public static void startMetaStores() {
*/
public static void startMetaStores(Map<MetastoreConf.ConfVars, String>
msConf,
Map<String, String> extraConf) {
+ // The Remote mini-HMS flips the static HiveMetaStore.isMetaStoreRemote
flag for the whole JVM, so the
+ // proxy-privilege check (authorizeProxyPrivilege) would also apply to the
Embedded variant and to the
+ // test user. Disable it unless a test explicitly sets it.
+ Map<String, String> effectiveExtraConf = new HashMap<>(extraConf);
+
effectiveExtraConf.putIfAbsent(MetastoreConf.ConfVars.EVENT_DB_NOTIFICATION_API_AUTH.getVarname(),
"false");
Review Comment:
any test is suffered without this property?
--
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]
