Re: Review Request 48494: Implement config values trimming for deployment via blueprint

2016-06-15 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48494/ --- (Updated Июнь 15, 2016, 2 п.п.) Review request for Ambari, Robert Nettleton, Se

Re: Review Request 48494: Implement config values trimming for deployment via blueprint

2016-06-15 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48494/#review137731 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 48494: Implement config values trimming for deployment via blueprint

2016-06-15 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48494/ --- (Updated Июнь 15, 2016, 1:30 п.п.) Review request for Ambari, Robert Nettleton,

Re: Review Request 48494: Implement config values trimming for deployment via blueprint

2016-06-15 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48494/#review137716 --- ambari-server/src/main/java/org/apache/ambari/server/controller/i

Re: Review Request 48494: Implement config values trimming for deployment via blueprint

2016-06-15 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48494/#review137714 --- Ship it! Ship It! - Sebastian Toader On June 14, 2016, 3:45

Re: Review Request 48494: Implement config values trimming for deployment via blueprint

2016-06-14 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48494/ --- (Updated Июнь 14, 2016, 1:45 п.п.) Review request for Ambari, Robert Nettleton,

Re: Review Request 48494: Implement config values trimming for deployment via blueprint

2016-06-10 Thread Dmytro Sen
> On Июнь 9, 2016, 4:05 п.п., Daniel Gergely wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java, > > line 343 > > > > > > Please consider cycloma

Re: Review Request 48494: Implement config values trimming for deployment via blueprint

2016-06-09 Thread Daniel Gergely
> On jún. 9, 2016, 4:05 du, Daniel Gergely wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java, > > line 343 > > > > > > Please consider cyclomati

Re: Review Request 48494: Implement config values trimming for deployment via blueprint

2016-06-09 Thread Dmytro Sen
> On Июнь 9, 2016, 4:26 п.п., Sebastian Toader wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java, > > line 343 > > > > > > Maybe we should think

Re: Review Request 48494: Implement config values trimming for deployment via blueprint

2016-06-09 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48494/#review136832 --- ambari-server/src/main/java/org/apache/ambari/server/controller/i

Re: Review Request 48494: Implement config values trimming for deployment via blueprint

2016-06-09 Thread Dmytro Sen
> On Июнь 9, 2016, 4:05 п.п., Daniel Gergely wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java, > > line 343 > > > > > > Please consider cycloma

Re: Review Request 48494: Implement config values trimming for deployment via blueprint

2016-06-09 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48494/#review136823 --- ambari-server/src/main/java/org/apache/ambari/server/controller/i