Github user rfarivar commented on a diff in the pull request:

    https://github.com/apache/storm/pull/927#discussion_r47553706
  
    --- Diff: external/storm-kafka/README.md ---
    @@ -226,8 +226,9 @@ You can return a null and the message will be ignored. 
If you have one static to
     DefaultTopicSelector.java and set the name of the topic in the constructor.
     
     ### Specifying Kafka producer properties
    -You can provide all the produce properties in your Storm topology by 
calling `KafkaBolt.withProducerProperties()` and 
`TridentKafkaStateFactory.withProducerProperties()`. Please see  
http://kafka.apache.org/documentation.html#newproducerconfigs
    -Section "Important configuration properties for the producer" for more 
details.
    +You can provide all the produce properties , see 
http://kafka.apache.org/documentation.html#producerconfigs 
    --- End diff --
    
    properties , see http:      -- >   properties, see http: 
    
    Remove space before comma


---
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.
---

Reply via email to