Github user jianbzhou commented on the pull request:

    https://github.com/apache/storm/pull/1131#issuecomment-218453241
  
    HI Hcml, 
    Just fyi - 
    In method doSeekRetriableTopicPartitions, we find below code:
    **kafkaConsumer.seekToEnd(rtp);    // Seek to last committed offset**
    Above code is contradictory with the comments, we replaced above line to 
below:
    **OffsetAndMetadata commitOffset = kafkaConsumer.committed(rtp);
    kafkaConsumer.seek(rtp, commitOffset.offset());    // Seek to last 
committed offset**
    Any comments please let me know. Thanks!
    
    For all above identified issues, we applied some quick and dirty fix and 
the testing is in progress, we will let you know the final testing result later.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to