Nick Allen

Hey @merrimanr - We don't need the following changes on this PR. I know these are just due to the iterations and changes I went through on #1515. * metron-platform/metron-indexing/metron-indexing-storm/pom.xml...

@kevin91nl Thanks for the submission! I really do appreciate the clarity of your code and explanations. > Performance A ChainParser does not need to interpret a language the way Stellar...

> @kevin91nl said... Stateful parsing: In one of our to-be-written parsers, we need to read an external file, parse the file and keep the parsed file in memory. As far...

@kevin91nl I do think that you've introduced a really useful concept though; chained parsers. I did not want to leave that unsaid. Your points about testing and sharing configuration are...

@kevin91nl I think the discussion that you have helped spark here is worthy of a separate email discuss thread. This is going to take more thought and discussion in the...

@aurelienduarte Thanks for the contribution. We need a few things to be able to review this. 1. Can you better describe the problem that you are solving with this patch?...

I have a [theory as to why this is happening](https://github.com/apache/metron/pull/1045#issuecomment-394033676) that should be fixed thanks to #1045. That being said, I am not sure that we need a `storm kill...

You should create a new JIRA not reuse 2088 please.

Here is what I am noticing with this change. If I both build the code and create the RPMs, I see times of roughly 11 - 12 minutes for RPM...

I ran this up in the dev environment and it looks good. Will be interested in @MohanDV 's feedback too.