sijie commented on a change in pull request #6052: [PIP-54] Acknowledgement for 
batch message local index
URL: https://github.com/apache/pulsar/pull/6052#discussion_r379160855
 
 

 ##########
 File path: managed-ledger/src/main/proto/MLDataFormats.proto
 ##########
 @@ -78,6 +81,11 @@ message MessageRange {
     required NestedPositionInfo upperEndpoint = 2;
 }
 
+message BatchDeletedIndexInfo {
 
 Review comment:
   The name is not very clear. I think it is keeping the deletion index for a 
batched entry. So I would suggest naming it "BatchedEntryDeletionIndexInfo"

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to