merlimat commented on a change in pull request #1066: Issue 937: add
CommandGetLastMessageId to make reader know the end of topic
URL: https://github.com/apache/incubator-pulsar/pull/1066#discussion_r167336300
##########
File path: pulsar-common/src/main/proto/PulsarApi.proto
##########
@@ -135,6 +135,7 @@ enum ProtocolVersion {
v9 = 9; // Added end of topic notification
v10 = 10;// Added proxy to broker
v11 = 11;// C++ consumers before this version are not correctly
handling the checksum field
+ //Added get topic's last messageId from broker
Review comment:
@zhaijack There was a merge issue here, `v11` was already taken in master
and this PR should be adding `v12` now.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
