Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2537#discussion_r165850084
--- Diff:
external/storm-kafka-client/src/main/java/org/apache/storm/kafka/spout/KafkaSpoutConfig.java
---
@@ -210,23 +215,26 @@ public Builder(String bootstrapServers, Subscription
subscription) {
}
/**
- * Set a {@link KafkaConsumer} property.
+ * Set a {@link KafkaConsumer} property. Please don't set
enable.auto.commit, instead set the {@link ProcessingGuarantee}
--- End diff --
Moved it to the documentation
---