saihemanth-cloudera commented on code in PR #4820:
URL: https://github.com/apache/hive/pull/4820#discussion_r1411148949


##########
standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/conf/MetastoreConf.java:
##########
@@ -1264,6 +1264,9 @@ public enum ConfVars {
         "hive.notification.sequence.lock.retry.sleep.interval", 10, 
TimeUnit.SECONDS,
         "Sleep interval between retries to acquire a notification lock as 
described part of property "
             + NOTIFICATION_SEQUENCE_LOCK_MAX_RETRIES.name()),
+    
NOTIFICATION_ALTER_PARTITIONS_V2_ENABLED("metastore.alterPartitions.notification.v2.enabled",
+        "hive.metastore.alterPartitions.notification.v2.enabled", true,

Review Comment:
   Should we disable this property by default? Whenever the consumers of these 
events are ready to process them, then they can enable it. We can probably 
enable this by default in Hive 5.0 release.



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