Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/2027#discussion_r107842202
--- Diff:
external/storm-kafka-client/src/main/java/org/apache/storm/kafka/spout/trident/KafkaTridentSpoutEmitter.java
---
@@ -149,7 +150,7 @@ private void emitTuples(TridentCollector collector,
ConsumerRecords<K, V> record
/**
* Determines the offset of the next fetch. For failed batches
lastBatchMeta is not null and contains the fetch
- * offset of the failed batch. In this scenario the next fetch will
take place at the offset of the failed batch.
+ * offset of the failed batch. In this scenario the next fetch will
take place at the offset of the failed batch + 1.
--- End diff --
minor nit: Just to make it clear.
>next fetch will take place at (offset of the failed batch + 1).
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
