Re: Review Request 51818: YAML Maps For Storm Are Not Being Escaped Correctly

2016-09-13 Thread Jonathan Hurley
> On Sept. 13, 2016, 8:48 a.m., Nate Cole wrote: > > ambari-common/src/main/python/ambari_commons/yaml_utils.py, lines 22-28 > > > > > > NOW I know who to go to for regex shenanigans ;) I always say, "If you need to

Re: Review Request 51818: YAML Maps For Storm Are Not Being Escaped Correctly

2016-09-13 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51818/#review148708 --- Ship it! ambari-common/src/main/python/ambari_commons/yaml_uti

Re: Review Request 51818: YAML Maps For Storm Are Not Being Escaped Correctly

2016-09-12 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51818/#review148582 --- Ship it! Ship It! - Robert Levas On Sept. 12, 2016, 3:13 p.m

Re: Review Request 51818: YAML Maps For Storm Are Not Being Escaped Correctly

2016-09-12 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51818/#review148562 --- Ship it! Ship It! - Alejandro Fernandez On Sept. 12, 2016, 7

Review Request 51818: YAML Maps For Storm Are Not Being Escaped Correctly

2016-09-12 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51818/ --- Review request for Ambari, Nate Cole and Robert Levas. Bugs: AMBARI-18366 h