merrimanr commented on issue #1403: METRON-2109: Add option to use Metron GUID as the id in Elasticsearch URL: https://github.com/apache/metron/pull/1403#issuecomment-494156784 The latest commit should address all of your feedback so far. For read-side testing I was able to add this to the `ElasticsearchSearchIntegrationTest` with only a couple of changes `ElasticSearchComponent` class. The test now includes bro messages with and without the Metron GUID. Does that provide the coverage you're looking for?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
