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


---

Reply via email to