Re: Review Request 62095: Installation should ignore OS that are not managed by Ambari

2017-09-05 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62095/#review184611 --- Fix it, then Ship it! Ship It!

Re: Review Request 62097: AMBARI-21871 : Add new graphs to HBase RegionServer dashboard in Grafana

2017-09-05 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62097/#review184583 --- Ship it! Ship It! - Sid Wagle On Sept. 5, 2017, 9:16 p.m.,

Review Request 62097: AMBARI-21871 : Add new graphs to HBase RegionServer dashboard in Grafana

2017-09-05 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62097/ --- Review request for Ambari, Dmytro Sen, Jayush Luniya, Sid Wagle, and Vivek

Review Request 62095: Installation should ignore OS that are not managed by Ambari

2017-09-05 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62095/ --- Review request for Ambari, Dmytro Grinenko and Jonathan Hurley. Bugs:

Re: Review Request 61998: Update Ambari Metrics service check to support HTTP SPNEGO authentication

2017-09-05 Thread Qin Liu
> On Aug. 31, 2017, 10:23 p.m., Eric Yang wrote: > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/service_check.py > > Lines 80 (patched) > > > > > > I would recommend to

Re: Review Request 61998: Update Ambari Metrics service check to support HTTP SPNEGO authentication

2017-09-05 Thread Qin Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61998/ --- (Updated Sept. 5, 2017, 7:50 p.m.) Review request for Ambari, Aravindan

Re: Review Request 61998: Update Ambari Metrics service check to support HTTP SPNEGO authentication

2017-09-05 Thread Qin Liu
> On Sept. 1, 2017, 12:41 p.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/service_check.py > > Lines 182-190 (patched) > > > > > > This can be

Re: Review Request 62072: Remove hive.custom-extensions.root from Hive Properties.

2017-09-05 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62072/#review184574 --- Ship it! Ship It! - Sid Wagle On Sept. 5, 2017, 6:46 p.m.,

Re: Review Request 62072: Remove hive.custom-extensions.root from Hive Properties.

2017-09-05 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62072/ --- (Updated Вер. 5, 2017, 6:46 після полудня) Review request for Ambari, Andrew

Re: Review Request 62072: Remove hive.custom-extensions.root from Hive Properties.

2017-09-05 Thread Sid Wagle
> On Sept. 5, 2017, 5:18 p.m., Sid Wagle wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog260.java > > Lines 379 (patched) > > > > > > Why is this done during Ambari upgrade as

Re: Review Request 62083: Fix Log Search upgrade to 3.0.0

2017-09-05 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62083/#review184569 --- Ship it! Ship It! - Robert Nettleton On Sept. 5, 2017, 3:52

Re: Review Request 61886: AMBARI-21803. Implement STOMP endpoint for alert definitions

2017-09-05 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61886/ --- (Updated Sept. 5, 2017, 8:20 p.m.) Review request for Ambari, Andrew Onischuk,

Re: Review Request 62083: Fix Log Search upgrade to 3.0.0

2017-09-05 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62083/#review184568 --- Ship it! Ship It! - Oliver Szabo On Sept. 5, 2017, 3:52

Re: Review Request 62072: Remove hive.custom-extensions.root from Hive Properties.

2017-09-05 Thread Vitalyi Brodetskyi
> On Вер. 5, 2017, 5:18 після полудня, Sid Wagle wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog260.java > > Lines 379 (patched) > > > > > > Why is this done during Ambari

Re: Review Request 62034: Integrate LogSearch new UI with the server and get rid of the old one

2017-09-05 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62034/#review184561 --- Ship it! Ship It! - Robert Nettleton On Sept. 5, 2017, 3:29

Re: Review Request 62072: Remove hive.custom-extensions.root from Hive Properties.

2017-09-05 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62072/#review184559 ---

Review Request 62084: Throw an error if unsupported database JDBC driver is configured for HDP services

2017-09-05 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62084/ --- Review request for Ambari, Attila Doroszlai, Attila Magyar, and Robert Levas.

Re: Review Request 62085: Upgrade to Ambari 2.6.0 fails with DB constraint violation on fk_sc_history_upgrade_id

2017-09-05 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62085/ --- (Updated Sept. 5, 2017, 7:25 p.m.) Review request for Ambari, Jonathan Hurley

Review Request 62085: Upgrade to Ambari 2.6.0 fails with DB constraint violation on fk_sc_history_upgrade_id

2017-09-05 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62085/ --- Review request for Ambari, Jonathan Hurley and Nate Cole. Bugs: AMBARI-21883

Review Request 62083: Fix Log Search upgrade to 3.0.0

2017-09-05 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62083/ --- Review request for Ambari, Oliver Szabo and Robert Nettleton. Bugs:

Re: Review Request 61886: AMBARI-21803. Implement STOMP endpoint for alert definitions

2017-09-05 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61886/#review184545 --- Atilla, as far as I saw from our test cluster, the java code you

Re: Review Request 62034: Integrate LogSearch new UI with the server and get rid of the old one

2017-09-05 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62034/ --- (Updated Sept. 5, 2017, 3:29 p.m.) Review request for Ambari, Andrii

Re: Review Request 62072: Remove hive.custom-extensions.root from Hive Properties.

2017-09-05 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62072/ --- (Updated Вер. 5, 2017, 3:06 після полудня) Review request for Ambari, Andrew

Re: Review Request 62072: Remove hive.custom-extensions.root from Hive Properties.

2017-09-05 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62072/ --- (Updated Вер. 5, 2017, 2:49 після полудня) Review request for Ambari, Andrew

Re: Review Request 62058: Deploys failing with MR service check failure due to missing mapreduce.tar.gz

2017-09-05 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62058/ --- (Updated Sept. 5, 2017, 2:07 p.m.) Review request for Ambari, Di Li and

Re: Review Request 62058: Deploys failing with MR service check failure due to missing mapreduce.tar.gz

2017-09-05 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62058/#review184541 --- Ship it! Ship It! - Vitalyi Brodetskyi On Вер. 4, 2017,

Re: Review Request 62081: AMBARI-21878. Implement STOMP endpoint to handle alerts from agent

2017-09-05 Thread Andrew Onischuk
> On Sept. 5, 2017, 12:44 p.m., Andrew Onischuk wrote: > > ambari-agent/src/main/python/ambari_agent/alerts/base_alert.py > > Line 149 (original), 149 (patched) > > > > > > Can we remove this line? > > Attila

Re: Review Request 62081: AMBARI-21878. Implement STOMP endpoint to handle alerts from agent

2017-09-05 Thread Attila Doroszlai
> On Sept. 5, 2017, 2:44 p.m., Andrew Onischuk wrote: > > ambari-agent/src/main/python/ambari_agent/alerts/base_alert.py > > Line 149 (original), 149 (patched) > > > > > > Can we remove this line? Sure. Should I

Re: Review Request 62081: AMBARI-21878. Implement STOMP endpoint to handle alerts from agent

2017-09-05 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62081/#review184532 --- ambari-agent/src/main/python/ambari_agent/alerts/base_alert.py

Review Request 62081: AMBARI-21878. Implement STOMP endpoint to handle alerts from agent

2017-09-05 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62081/ --- Review request for Ambari, Andrew Onischuk, Myroslav Papirkovskyy, Sandor

Re: Review Request 61886: AMBARI-21803. Implement STOMP endpoint for alert definitions

2017-09-05 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61886/ --- (Updated Sept. 5, 2017, 2:39 p.m.) Review request for Ambari, Andrew Onischuk,

Re: Review Request 52785: Fix problem on Suse for ambari-agent installation

2017-09-05 Thread Attila Doroszlai
> On Oct. 13, 2016, 2:26 p.m., Attila Doroszlai wrote: > > ambari-server/src/main/python/setupAgent.py > > Line 125 (original), 125 (patched) > > > > > > The `-y` flag for `install` is an alias for

Re: Review Request 52785: Fix problem on Suse for ambari-agent installation

2017-09-05 Thread Zsombor Gegesy
> On Oct. 13, 2016, 12:26 p.m., Attila Doroszlai wrote: > > ambari-server/src/main/python/setupAgent.py > > Line 125 (original), 125 (patched) > > > > > > The `-y` flag for `install` is an alias for

Re: Review Request 52785: Fix problem on Suse for ambari-agent installation

2017-09-05 Thread Attila Doroszlai
> On Oct. 13, 2016, 2:26 p.m., Attila Doroszlai wrote: > > ambari-server/src/main/python/setupAgent.py > > Line 125 (original), 125 (patched) > > > > > > The `-y` flag for `install` is an alias for

Re: Review Request 52785: Fix problem on Suse for ambari-agent installation

2017-09-05 Thread Zsombor Gegesy
> On Oct. 13, 2016, 12:26 p.m., Attila Doroszlai wrote: > > ambari-server/src/main/python/setupAgent.py > > Line 125 (original), 125 (patched) > > > > > > The `-y` flag for `install` is an alias for

Re: Review Request 62064: Support for handling ldap admin group mappings

2017-09-05 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62064/ --- (Updated Sept. 5, 2017, 9:26 a.m.) Review request for Ambari, Attila

Re: Review Request 62064: Support for handling ldap admin group mappings

2017-09-05 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62064/ --- (Updated Sept. 5, 2017, 8:30 a.m.) Review request for Ambari, Attila

Re: Review Request 62064: Support for handling ldap admin group mappings

2017-09-05 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62064/#review184512 --- Fix it, then Ship it!