Github user mmiklavc commented on the issue:
https://github.com/apache/metron/pull/863
@ottobackwards created this to track it -
https://issues.apache.org/jira/browse/METRON-1524
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/863
I think this is a mess that we will need to sort out, but I won't hold this
up for that.
I'm +0 I guess.
---
Github user mmiklavc commented on the issue:
https://github.com/apache/metron/pull/863
It seems this PR is meant to address the global case, since the writer is
general purpose. Per @cestella's comments, I don't know that we can include
original_string. I do think there's value in min
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/863
What I think would be better would be to have a readme about the required
fields and refer to that document from these other documents
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/863
The parser topology PRODUCES the original_string. By that very action, and
it's requirement, it can only be used and required by downstream usages either
from rest ( post indexing ) or during
Github user justinleet commented on the issue:
https://github.com/apache/metron/pull/863
@mmiklavc @cestella @ottobackwards Is this good to be merged in?
---