Github user mmiklavc commented on the issue:
https://github.com/apache/metron/pull/1213
@ottobackwards - @merrimanr and I had a chat about this offline and
realized there was another alternative to what we had come up with in the
multiline grok and syslog PRs. Basically, just making the existing methods in
MessageParser `@deprecated` and with a new default implementation for
`parse(..)` that throws a `NotImplementedException`. This does clean up some of
the shrapnel we had discussed on 1234 such that there's no further need for a
decorator.
@merrimanr - the only thing missing here I think is to mark the `parse` and
`parseOptional` methods `@deprecated`. Also, I didn't see a delete of
`MultilineMessageParser` in your latest commits to go with that refactor/merge.
We probably want to get rid of it altogether if we take this alternate
approach. Also, I think we should make a more public statement on the user and
dev list about deprecating the methods if we all agree on this approach to give
users time to migrate existing custom parser implementations.
---
