Github user hmcl commented on a diff in the pull request:
https://github.com/apache/storm/pull/2380#discussion_r147013385
--- Diff:
external/storm-kafka-client/src/main/java/org/apache/storm/kafka/spout/KafkaSpout.java
---
@@ -78,17 +78,17 @@
private transient KafkaSpoutRetryService retryService;
// Handles tuple events (emit, ack etc.)
private transient KafkaTupleListener tupleListener;
- // timer == null for modes other than at-least-once
+ // timer == null if processing guarantee is other than at-least-once
private transient Timer commitTimer;
// Flag indicating that the spout is still undergoing initialization
process.
private transient boolean initialized;
// Initialization is only complete after the first call to
KafkaSpoutConsumerRebalanceListener.onPartitionsAssigned()
// Tuples that were successfully acked/emitted. These tuples will be
committed periodically when the commit timer expires,
- //or after a consumer rebalance, or during close/deactivate. Always
empty if not using at-least-once mode.
+ // or after a consumer rebalance, or during close/deactivate. Always
empty if not using at-least-once processing guarantee.
--- End diff --
Done. Removed the double negation.
---
