GitHub user cestella opened a pull request:

    https://github.com/apache/incubator-metron/pull/143

    METRON-197: Validation should be the last step in the ParserBolt

    Right now we are doing the validation prior to the messageFilter.  We 
should only validate the parsed messages which passes through the filter.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/cestella/incubator-metron METRON-197

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-metron/pull/143.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #143
    
----
commit 53e3f61fe795b1620ee5847448ba1a2a13274120
Author: cstella <ceste...@gmail.com>
Date:   2016-06-03T02:25:16Z

    Fixing global validation to run at the end of the validation pipeline.

commit 134e4cd02a149452f9f774a6a85d6c7ccefabc17
Author: cstella <ceste...@gmail.com>
Date:   2016-06-03T03:07:24Z

    Fixing ParserBolt to do validations in the proper place.

----


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to