Re: Review Request 62294: Batch requests(Rolling Restart, HSI enable/disable, decommission/recommission) not working when Ambari is configured with Knox proxy

2017-09-13 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62294/#review185358 --- Ship it! Ship It! - Sid Wagle On Sept. 13, 2017, 10:54

Re: Review Request 62294: Batch requests(Rolling Restart, HSI enable/disable, decommission/recommission) not working when Ambari is configured with Knox proxy

2017-09-13 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62294/ --- (Updated Sept. 13, 2017, 10:54 p.m.) Review request for Ambari, Myroslav

Re: Review Request 62289: Package Installation is failing If Patch-VDF contain service that is not installed in the cluster

2017-09-13 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62289/#review185343 --- Ship it! Ship It! - Jonathan Hurley On Sept. 13, 2017, 2:25

Re: Review Request 60358: Hivesevers's status shows issue for the thrift's port conflict, when hiveservers and spark thrift server installed in the same host.

2017-09-13 Thread Nate Cole
> On Sept. 12, 2017, 10:23 a.m., Nate Cole wrote: > > What is the status of this review? Ping - Nate --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60358/#review185187

Re: Review Request 62294: Batch requests(Rolling Restart, HSI enable/disable, decommission/recommission) not working when Ambari is configured with Knox proxy

2017-09-13 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62294/#review185336 --- Fix it, then Ship it!

Re: Review Request 62291: Last Run Service Check for SPARK Picks up SPARK2

2017-09-13 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62291/#review185337 --- Ship it! Ship It! - Nate Cole On Sept. 13, 2017, 3:15 p.m.,

Review Request 62294: Batch requests(Rolling Restart, HSI enable/disable, decommission/recommission) not working when Ambari is configured with Knox proxy

2017-09-13 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62294/ --- Review request for Ambari, Myroslav Papirkovskyy, Sid Wagle, and Yusaku Sako.

Re: Review Request 62292: AMBARI-21952 : Refine AMS heap recommendations in stack advisor.

2017-09-13 Thread Aravindan Vijayan
> On Sept. 13, 2017, 7:56 p.m., Sid Wagle wrote: > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py > > Lines 873 (patched) > > > > > > Is this left behind artifact? Yes, just a missed

Re: Review Request 62292: AMBARI-21952 : Refine AMS heap recommendations in stack advisor.

2017-09-13 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62292/#review185333 --- Ship it!

Re: Review Request 62290: Validate kerberos.json files to ensure they meet the expected schema

2017-09-13 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62290/ --- (Updated Sept. 13, 2017, 3:52 p.m.) Review request for Ambari, Attila Magyar,

Re: Review Request 62292: AMBARI-21952 : Refine AMS heap recommendations in stack advisor.

2017-09-13 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62292/ --- (Updated Sept. 13, 2017, 7:49 p.m.) Review request for Ambari, Dmytro Sen,

Review Request 62292: AMBARI-21952 : Refine AMS heap recommendations in stack advisor.

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

Re: Review Request 62291: Last Run Service Check for SPARK Picks up SPARK2

2017-09-13 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62291/#review185329 ---

Review Request 62291: Last Run Service Check for SPARK Picks up SPARK2

2017-09-13 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62291/ --- Review request for Ambari, Nate Cole and Robert Levas. Bugs: AMBARI-21951

Review Request 62290: Validate kerberos.json files to ensure they meet the expected schema

2017-09-13 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62290/ --- Review request for Ambari, Attila Magyar, Balázs Bence Sári, Eugene Chekanskiy,

Re: Review Request 62242: AMBARI-21933 UI: Implement breadcrumbs in Background Operations modal

2017-09-13 Thread Jason Golieb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62242/ --- (Updated Sept. 13, 2017, 6:31 p.m.) Review request for Ambari, Denys Buzhor,

Review Request 62289: Package Installation is failing If Patch-VDF contain service that is not installed in the cluster

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

Re: Review Request 62245: Preview. Auto fix enhancement to remove more than 1 selected configs

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

Re: Review Request 62259: Upgrade to IOP 4.2.5 from IOP 4.1 failed with combined Solr host names longer than item_text column size in table upgrade_item

2017-09-13 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62259/#review185308 --- Ship it! Ship It! - Robert Levas On Sept. 12, 2017, 9:03

Re: Review Request 62259: Upgrade to IOP 4.2.5 from IOP 4.1 failed with combined Solr host names longer than item_text column size in table upgrade_item

2017-09-13 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62259/#review185307 --- Ship it! Ship It! - Nate Cole On Sept. 12, 2017, 9:03 p.m.,

Re: Review Request 62258: Stack selection page does not load the HDP stacks [Intermittent]

2017-09-13 Thread Nate Cole
> On Sept. 13, 2017, 10:35 a.m., Dmitro Lisnichenko wrote: > > ambari-server/src/main/java/org/apache/ambari/server/stack/StackContext.java > > Line 142 (original), 150 (patched) > > > > > > Maybe ConcurrentHashMap

Re: Review Request 62258: Stack selection page does not load the HDP stacks [Intermittent]

2017-09-13 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62258/#review185295 ---

Re: Review Request 62245: Preview. Auto fix enhancement to remove more than 1 selected configs

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

Re: Review Request 62258: Stack selection page does not load the HDP stacks [Intermittent]

2017-09-13 Thread Nate Cole
> On Sept. 13, 2017, 9:21 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/stack/StackContext.java > > Lines 204 (patched) > > > > > > Should we log a warning here (not

Re: Review Request 62258: Stack selection page does not load the HDP stacks [Intermittent]

2017-09-13 Thread Nate Cole
> On Sept. 12, 2017, 5:14 p.m., Dmytro Grinenko wrote: > > ambari-server/src/main/java/org/apache/ambari/server/stack/StackContext.java > > Line 151 (original), 159 (patched) > > > > > > can we move amount of

Re: Review Request 62258: Stack selection page does not load the HDP stacks [Intermittent]

2017-09-13 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62258/#review185284 --- Fix it, then Ship it!

Re: Review Request 62277: Log Search UI: reorganize generated files structure

2017-09-13 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62277/#review185283 --- Ship it! Ship It! - Aleksandr Kovalenko On Сен. 13, 2017,

Re: Review Request 62277: Log Search UI: reorganize generated files structure

2017-09-13 Thread Andrii Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62277/ --- (Updated Вер. 13, 2017, 1:16 після полудня) Review request for Ambari and

Re: Review Request 62277: Log Search UI: reorganize generated files structure

2017-09-13 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62277/#review185281 --- Ship it! Ship It! - Aleksandr Kovalenko On Сен. 13, 2017,

Re: Review Request 62277: Log Search UI: reorganize generated files structure

2017-09-13 Thread Andrii Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62277/ --- (Updated Вер. 13, 2017, 12:06 після полудня) Review request for Ambari and

Re: Review Request 62278: Ambari updates database/tables location from intended remote HDFS to local HDFS

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

Re: Review Request 62277: Log Search UI: reorganize generated files structure

2017-09-13 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62277/#review185278 --- Ship it! Ship It! - Aleksandr Kovalenko On Сен. 13, 2017,

Review Request 62277: Log Search UI: reorganize generated files structure

2017-09-13 Thread Andrii Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62277/ --- Review request for Ambari and Aleksandr Kovalenko. Bugs: AMBARI-21946

Review Request 62278: Ambari updates database/tables location from intended remote HDFS to local HDFS

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

Re: Review Request 61898: Create Utility Script to support Solr Collection Data Retention/Purging/Archiving

2017-09-13 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61898/#review185274 --- Ship it! Ship It! - Oliver Szabo On Sept. 12, 2017, 3:30

Re: Review Request 62182: Ambari is setting "atlas.graph.storage.hostname" to hbase hostname, instead it should set the zookeeper quorum.

2017-09-13 Thread Sebastian Toader
> On Sept. 12, 2017, 5:09 p.m., Ayub Pathan wrote: > > I dont see UTs added for BlueprintConfigurationProcessor.java. But the > > patch attached to the jira has the tests changes as well. Please check. Thanks for pointing it out. I uploaded the wrong patch. - Sebastian