Re: Review Request 51685: AMBARI-18328: Blueprints: Log "setting" section of blueprint in ambari server log file

2016-09-07 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51685/ --- (Updated Sept. 7, 2016, 12:48 p.m.) Review request for Ambari, Robert

Re: Review Request 51685: AMBARI-18328: Blueprints: Log "setting" section of blueprint in ambari server log file

2016-09-07 Thread Robert Nettleton
> On Sept. 7, 2016, 1:58 p.m., Robert Nettleton wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintResourceProvider.java, > > line 499 > > > > > > Why is this being logged?

Re: Review Request 51685: AMBARI-18328: Blueprints: Log "setting" section of blueprint in ambari server log file

2016-09-07 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51685/ --- (Updated Sept. 7, 2016, 12:29 p.m.) Review request for Ambari, Robert

Re: Review Request 51685: AMBARI-18328: Blueprints: Log "setting" section of blueprint in ambari server log file

2016-09-07 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51685/#review148010 --- Fix it, then Ship it! The change looks fine to me, just need a

Re: Review Request 51685: AMBARI-18328: Blueprints: Log "setting" section of blueprint in ambari server log file

2016-09-06 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51685/#review147964 --- Ship it! Ship It! - Sumit Mohanty On Sept. 7, 2016, 2:14