[GitHub] justinleet commented on issue #1325: METRON-1970 Add Metadata to Error Messages Generated During Parsing

[GitBox <gi...@apache.org>]

justinleet commented on issue #1325: METRON-1970 Add Metadata to Error Messages Generated During Parsing
URL: https://github.com/apache/metron/pull/1325#issuecomment-457708817
 
 
   I believe we index errors into ES and Solr, right? I would expect the metadata fields to not show up in Solr (we have a dynamic catch-all that says to ignore anything we don't know about: [schema.xml](https://github.com/apache/metron/blob/bc16e68252de98846bba58aafb2d7d445caac9c1/metron-platform/metron-solr/src/main/config/schema/error/schema.xml#L45)).  If users are using the real time stores to sift through errors, I'm concerned they won't be able to.
   
   My full dev is being sketchy, so I'm trying to spin up ES now to double check what happens. I would expect ES to attempt to dynamically add the fields based on their types (which is probably fine, although it'd be nice to add the well understood fields).

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services