[GitHub] incubator-metron issue #365: METRON-565: apps/metron/enrichment/indexed dire...

2016-11-29 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/365 Yep, I'm +1, too. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] incubator-metron issue #365: METRON-565: apps/metron/enrichment/indexed dire...

2016-11-23 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/365 Ended up making a few additional changes here. First, got rid of the /apps/metron/enrichment directory entirely. With indexing being split off into its own topology, this no longer made

[GitHub] incubator-metron issue #365: METRON-565: apps/metron/enrichment/indexed dire...

2016-11-23 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/365 @mmiklavc Looks like https://github.com/apache/incubator-metron/pull/362 for METRON-575 got pulled in yesterday. Could you either kick this PR to get it to rerun and hopefully run

[GitHub] incubator-metron issue #365: METRON-565: apps/metron/enrichment/indexed dire...

2016-11-22 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/incubator-metron/pull/365 Yes, appears to be an intermittent timing problem. If you re-run, I assume it would pass. In METRON-575 I changed how these tests are validated. I assume that would help

[GitHub] incubator-metron issue #365: METRON-565: apps/metron/enrichment/indexed dire...

2016-11-21 Thread james-sirota
Github user james-sirota commented on the issue: https://github.com/apache/incubator-metron/pull/365 @nickwallen can you take a look and verify? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] incubator-metron issue #365: METRON-565: apps/metron/enrichment/indexed dire...

2016-11-21 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/365 The Travis failure appears to be an unrelated profiler integration test error, although I'd like @mmiklavc to double-check. Assuming it is unrelated, I'm +1, by inspection. Thanks for