[ 
https://issues.apache.org/jira/browse/METRON-1304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16679990#comment-16679990
 ] 

ASF GitHub Bot commented on METRON-1304:
----------------------------------------

Github user JonZeolla commented on the issue:

    https://github.com/apache/metron-bro-plugin-kafka/pull/2
  
    In looking through the PR, this is mostly related to the include/exclude 
functionality yes, but you are right that it may look like there are some other 
things included because:
    * As a part of updating the documentation for these new features I 
attempted to make things more uniform in format (which included some text 
reordering), and I added details regarding what a successful `bro-pkg` install 
looks like (before this PR that was broken/undocumented)
    * This also fixes the previously broken btest which was a 1 line change to 
`tests/Baseline/kafka.show-plugin/output`
    * There are some minor documentation changes such as changing suggested 
priority to -10, or adding multiple brokers to the docs in response to a 
[review 
comment](https://github.com/apache/metron-bro-plugin-kafka/pull/2/files#r224996990).


> Allow metron-bro-plugin-kafka to include or exclude logs
> --------------------------------------------------------
>
>                 Key: METRON-1304
>                 URL: https://issues.apache.org/jira/browse/METRON-1304
>             Project: Metron
>          Issue Type: Improvement
>            Reporter: Jon Zeolla
>            Assignee: Jon Zeolla
>            Priority: Major
>
> Right now, you must specify which logs you want to send to kafka via 
> metron-bro-plugin-kafka.  This would allow the additional feature of 
> excluding certain logs, and sending everything else.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to