dengzhhu653 commented on code in PR #4820:
URL: https://github.com/apache/hive/pull/4820#discussion_r1419874250


##########
hcatalog/server-extensions/src/main/java/org/apache/hive/hcatalog/listener/DbNotificationListener.java:
##########
@@ -481,6 +482,25 @@ public void onAlterPartition(AlterPartitionEvent 
partitionEvent) throws MetaExce
     process(event, partitionEvent);
   }
 
+  @Override
+  public void onAlterPartitions(AlterPartitionsEvent event) throws 
MetaException {
+    Table table = event.getTable();
+    Iterator<List<Partition>> iterator = 
event.getNewPartsIterator(maxBatchSize);

Review Comment:
   There already has a property 
`metastore.notification.parameters.exclude.patterns` for such purpose, 
   
https://github.com/apache/hive/blob/master/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/conf/MetastoreConf.java#L839-L844
   
   During serializing the partitions, 
   
https://github.com/apache/hive/pull/4820/files#diff-217ba1e72ae13548c09bd705eae25b302b80215031282171257569c05261c89cR77
   MessageBuilder#createPartitionObjJson will exclude the matched properties 
from the partition parameters,
   
https://github.com/apache/hive/blob/master/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/messaging/MessageBuilder.java#L411
   



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