ConcurrencyPractitioner commented on issue #3983: [Issue#3976] 
[component/client] Adding config for ReaderImpl
URL: https://github.com/apache/pulsar/pull/3983#issuecomment-480454749
 
 
   @sijie  I noticed that usually the user would either use startMessageId(long 
timestamp) or startMessageId(MessageId id) but not both. So should we make it 
so that we need to call only one of the methods to set the message id at which 
the user would need to start? Currently, it is required that MessageId id be a 
non-null parameter, so if the user wants to use startMessageId(long timestamp), 
they would also have to call startMessageId(MessageId id) with some dummy 
variable (id) as well.

----------------------------------------------------------------
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

Reply via email to