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


---

Reply via email to