Github user mmiklavc commented on the issue:
https://github.com/apache/metron/pull/839
+1 by inspection. Thanks @nickwallen!
---
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/839
+1 (non-binding). Validated again on full-dev. Works fine, @nickwallen !
---
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/839
I ran this through another full test after merge with master. Everything
looks good to me.
---
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/839
Thanks @anandsubbu . I resolved the merge conflicts from master.
---
Github user anandsubbu commented on the issue:
https://github.com/apache/metron/pull/839
+1 (non-binding) works fine, and a much needed fix as well ! If for some
reason, we were not able to install the templates during the startup, it would
be caught by the service check.
