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 2019/11/02 13:56:18 UTC

[GitHub] [pulsar] candlerb commented on issue #5541: Python API: seek(eventtime)

candlerb commented on issue #5541: Python API: seek(eventtime)
URL: https://github.com/apache/pulsar/issues/5541#issuecomment-549046106
 
 
   On further investigation I think some work on C++ API would be needed as well.
   
   1. C++ API has seek in Consumer but not in Reader (which is where I want it)
   2. `Consumer::seek` accepts only `MessageId` as its argument - although I see no reason why it can't be overloaded like its Java counterpart
   
   In terms of the wire protocol, it would need to copy whatever the Java client [does](https://github.com/apache/pulsar/blob/master/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java#L1386) for `seek(timestamp)` - the command is built [here](https://github.com/apache/pulsar/blob/master//pulsar-common/src/main/java/org/apache/pulsar/common/protocol/Commands.java#L563).
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services