Re: Review Request 62474: AMS: Add smart configs support for service

2017-09-21 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62474/#review185920 --- Ship it! Ship It! - Aravindan Vijayan On Sept. 21, 2017,

Re: Review Request 62474: AMS: Add smart configs support for service

2017-09-21 Thread Sid Wagle
> On Sept. 21, 2017, 7:37 p.m., Aravindan Vijayan wrote: > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/configuration/ams-env.xml > > Lines 81 (patched) > > > > > > Should we allow minimum

Re: Review Request 62474: AMS: Add smart configs support for service

2017-09-21 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62474/ --- (Updated Sept. 21, 2017, 7:57 p.m.) Review request for Ambari, Aravindan

Re: Review Request 62474: AMS: Add smart configs support for service

2017-09-21 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62474/#review185913 ---

Review Request 62474: AMS: Add smart configs support for service

2017-09-21 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62474/ --- Review request for Ambari, Aravindan Vijayan, Dmytro Sen, and Jaimin Jetly.