[GitHub] metron issue #1022: METRON-1569 Allow user to change field name conversion w...

2018-05-30 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/1022 +1 by inspection. This is great @nickwallen, thanks for the contribution. ---

[GitHub] metron issue #1022: METRON-1569 Allow user to change field name conversion w...

2018-05-29 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/1022 Ok, I'm comfortable with the change now. +1 by inspection, pending @mmiklavc ---

[GitHub] metron issue #1022: METRON-1569 Allow user to change field name conversion w...

2018-05-29 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/1022 All review feedback should have been addressed. Let me know if there is anything else @mmiklavc @cestella ---

[GitHub] metron issue #1022: METRON-1569 Allow user to change field name conversion w...

2018-05-25 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/1022 FYI - I also confirmed that the configuration value can be changed using the Advanced mode in the Management UI. ![screen shot 2018-05-25 at 11 31 17