Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/1191
Thanks! This has been merged into feature/METRON-1699-create-batch-profiler
---
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/1191
That being said, I will own opening a proper discuss thread on the mailing
list to see if there are other features (like Mpack integration) that I should
take care of before merging this feature
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/1191
Thanks @justinleet.
> @justinleet: ... should the default batch profiler config live in Ambari?
That's the feedback I am looking for on the FB, so thanks.
Just to lev
Github user justinleet commented on the issue:
https://github.com/apache/metron/pull/1191
+1, I spun it up and ran through the tests. Thanks for the contribution
It's unrelated to getting this PR in (and probably not the right spot for
this question), but should the default b
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/1191
```
Failed tests:
ZKConfigurationsCacheIntegrationTest.validateUpdate:230->lambda$validateUpdate$9:230
expected:<{hdfs={index=yaf, batchSize=1, enabled=true},
elasticsearch={index=yaf
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/1191
After merging master, I just ran through the test steps again just to
re-validate.
---
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/1191
Crank. Ci failure because...
> The job exceeded the maximum time limit for jobs, and has been terminated.
---
