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

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!

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

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 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

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

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

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

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 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

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

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

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