You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/09/29 05:39:28 UTC

[GitHub] [pulsar] MarvinCai edited a comment on pull request #12032: [consumer] Add seek by index for consumer

MarvinCai edited a comment on pull request #12032:
URL: https://github.com/apache/pulsar/pull/12032#issuecomment-929850723


   But not all users will use Index or even BrokerEntryMetadata, while all messages have Timestamp and MessageID, that's a bit different in my opinion. Adding a method that rely on a property might not exist for all users is kind of weird. 


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org