zzzming 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_r378954804
##########
File path:
pulsar-common/src/main/java/org/apache/pulsar/common/api/proto/PulsarApi.java
##########
@@ -1209,11 +1214,26 @@ public int getBatchIndex() {
return batchIndex_;
}
+ // repeated int64 ackSet = 5;
+ public static final int ACKSET_FIELD_NUMBER = 5;
+ private java.util.List<java.lang.Long> ackSet_;
+ public java.util.List<java.lang.Long>
+ getAckSetList() {
+ return ackSet_;
+ }
+ public int getAckSetCount() {
+ return ackSet_.size();
+ }
+ public long getAckSet(int index) {
+ return ackSet_.get(index);
+ }
+
private void initFields() {
ledgerId_ = 0L;
entryId_ = 0L;
partition_ = -1;
batchIndex_ = -1;
+ ackSet_ = java.util.Collections.emptyList();;
Review comment:
Double `;;` You might want to do a global replacement since these are in
multiple places.
----------------------------------------------------------------
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
