[
https://issues.apache.org/jira/browse/METRON-1815?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16675559#comment-16675559
]
ASF GitHub Bot commented on METRON-1815:
----------------------------------------
Github user justinleet commented on the issue:
https://github.com/apache/metron/pull/1249
I think everyone's fine with splitting out metron-parsers and
metron-parsers-common. I think there's a reasonable argument to give people a
splitting out point from the basic parsers. I personally think it's a bit
overkill, but I'm more than happy to compromise on that.
I think the hangup right now is that in this structure
```
└── metron-parsing
├── metron-parsers
├── metron-parsers-common
├── metron-parsing-common
└── metron-parsing-storm
```
what actually lives in metron-parsing-common if we choose to create it? It
seems like the only thing that maybe fits in there is ParserRunner and
ParserRunnerImpl. Possibly a few other miscellaneous classes. I agree that,
logically speaking, we could split that out. I just don't think that we need to
split out such a small module. If, as we start implementing any new platforms,
there's enough to merit setting up a module, I think it's perfectly fine to do
it as part of that effort.
> Separate metron-parsers into metron-parsers-common and metron-parsers-storm
> ---------------------------------------------------------------------------
>
> Key: METRON-1815
> URL: https://issues.apache.org/jira/browse/METRON-1815
> Project: Metron
> Issue Type: Improvement
> Reporter: Justin Leet
> Assignee: Justin Leet
> Priority: Major
>
> In order to expose our parsers to 3rd party components (e.g. the discussions
> on NiFi and potentially other platforms like Spark), we should
> separate the storm-bits into its own project. The metron-parsers-common
> project should contain only parser-oriented code, whereas the
> metron-parsers-storm project should contain the storm specific code
> (e.g. the parser bolt).
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
