codelipenghui commented on a change in pull request #5491: Fix message
deduplicate issue while using external sequence id with batch produce
URL: https://github.com/apache/pulsar/pull/5491#discussion_r340939035
##########
File path: pulsar-common/src/main/proto/PulsarApi.proto
##########
@@ -394,6 +398,10 @@ message CommandSend {
optional int32 num_messages = 3 [default = 1];
optional uint64 txnid_least_bits = 4 [default = 0];
optional uint64 txnid_most_bits = 5 [default = 0];
+
+ /// Add lowest and highest sequence id to support external sequence id
+ optional uint64 lowest_sequence_id = 6 [default = 0];
+ optional uint64 highest_sequence_id = 7 [default = 0];
Review comment:
Yes, we can only add a new last_sequence_id, will fix.
----------------------------------------------------------------
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
