[GitHub] [metron-bro-plugin-kafka] mauropalumbo75 opened a new pull request #40: METRON-2309 Add a Kafka "metadata.broker.list" for each log writer filter.

[GitBox]

mauropalumbo75 opened a new pull request #40: METRON-2309 Add a Kafka 
"metadata.broker.list" for each log writer filter.
URL: https://github.com/apache/metron-bro-plugin-kafka/pull/40
 
 
    Hi,
      in examples 4-5 in readme.md file it seems like you can set a different 
kafka broker list for each different bro filter, as in:
   
   local http_filter: Log::Filter = [
           $name = "kafka-http",
           $writer = Log::WRITER_KAFKAWRITER,
           $config = table(
                   ["metadata.broker.list"] = "localhost:9092"
           ),
           $path = "http"
       ];
   
   However, looking at the code here, this is the case only for the topic_name. 
I fixed this.
   
   Mauro
   
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services