rdhabalia commented on a change in pull request #12403:
URL: https://github.com/apache/pulsar/pull/12403#discussion_r756407592
##########
File path: pulsar-common/src/main/proto/PulsarApi.proto
##########
@@ -61,6 +61,9 @@ message MessageIdData {
optional int32 batch_index = 4 [default = -1];
repeated int64 ack_set = 5;
optional int32 batch_size = 6;
+
+ // For the chunk message id, we need to specify the first chunk message id.
+ optional MessageIdData first_chunk_message_id = 7;
Review comment:
> And introducing the new chunk message ID will allow us to peek the
message from the topic if the application recorded the chunk message ID.
@codelipenghui issue is not with introducing chunk message Id or changes at
`MessageId`. the fundamental issue is this entire change is related get
`messageId` of first chunk into last chunk data. but that's not guaranteed, so,
what's the point of changes which is built on wrong assumption.
--
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]
