Review Request 63464: Beeline connection hangs for longer time when connection to HS2 with metastore DB down

2017-11-01 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63464/ --- Review request for Ambari and Vitalyi Brodetskyi. Bugs: AMBARI-22346 https:

Re: Review Request 63450: Improve KDC integration

2017-11-01 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63450/#review189799 --- Ship it! Ship It! - Robert Nettleton On Oct. 31, 2017, 9:03

Re: Review Request 63440: Metrics for OneFS mpack

2017-11-01 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63440/#review189803 --- contrib/management-packs/isilon-onefs-mpack/src/main/resources/ad

Re: Review Request 63051: Add service group dependencies

2017-11-01 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63051/#review189810 --- [~vbrodetskyi] Generally this API looks good. If we make the API t

Re: Review Request 63051: Add service group dependencies

2017-11-01 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63051/#review189812 --- ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql L

Re: Review Request 63051: Add service group dependencies

2017-11-01 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63051/#review189813 --- Ship it! Go ahead and commit this patch and make the suggested

Re: Review Request 63457: Add service dependencies

2017-11-01 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63457/#review189814 --- ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql L

Re: Review Request 63457: Add service dependencies

2017-11-01 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63457/#review189815 --- Ship it! Ship It! - Jayush Luniya On Nov. 1, 2017, 12:20 a.m

Re: Review Request 63441: Remove Slider from Ambari stack definition of HDP 3.0

2017-11-01 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63441/#review189818 --- This will break HIVE LLAP as it uses slider. Please don't make any

Re: Review Request 63205: AMBARI-22198. Implement read only API for getting stack level settings.

2017-11-01 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63205/#review189819 --- Ship it! Ship It! - Jayush Luniya On Oct. 25, 2017, 6:44 p.m

Review Request 63477: AMBARI-22348 : Metric Definition Service V1 Implementation.

2017-11-01 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63477/ --- Review request for Ambari, Jayush Luniya and Sid Wagle. Bugs: AMBARI-22348

Re: Review Request 63477: AMBARI-22348 : Metric Definition Service V1 Implementation.

2017-11-01 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63477/ --- (Updated Nov. 1, 2017, 9:21 p.m.) Review request for Ambari, Jayush Luniya and

Re: Review Request 63477: AMBARI-22348 : Metric Definition Service V1 Implementation.

2017-11-01 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63477/#review189837 --- Fix it, then Ship it! ambari-metrics/ambari-metrics-anomaly-de

Re: Review Request 63477: AMBARI-22348 : Metric Definition Service V1 Implementation.

2017-11-01 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63477/ --- (Updated Nov. 1, 2017, 10:37 p.m.) Review request for Ambari, Jayush Luniya and

Re: Review Request 63477: AMBARI-22348 : Metric Definition Service V1 Implementation.

2017-11-01 Thread Aravindan Vijayan
> On Nov. 1, 2017, 10:13 p.m., Sid Wagle wrote: > > ambari-metrics/ambari-metrics-anomaly-detection-service/src/main/scala/org/apache/ambari/metrics/adservice/metricmanager/MetricManagerServiceImpl.scala > > Lines 29 (patched) > >

Re: Review Request 62930: Non-root ambari user causes Express Upgrade to fail during checking HDFS safe mode phase

2017-11-01 Thread Piyush Chavhan
> On Oct. 30, 2017, 3:32 p.m., Jonathan Hurley wrote: > > Ship It! Thanks. - Piyush --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62930/#review189596 ---