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


##########
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:
   The JDBC max batch size may not be equal to Thrift max message size. So the 
consumers of these events may get thrift max message limit error if stats are 
stored in each of these partitions. So this is a bottle-neck to store the 
entire partition object. Can we do something else to satisfy the replication 
issue and also at the same time to bring down the size of serialized object?



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