Review Request 50101: Cluster user role is permitted to install packages using API

2016-07-15 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50101/ --- Review request for Ambari, Jonathan Hurley, Myroslav Papirkovskyy, Nate Cole,

Review Request 50100: AMBARI-17754 - Issue with modify configs for hive service

2016-07-15 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50100/ --- Review request for Ambari and Andriy Babiichuk. Bugs: AMBARI-17754

Review Request 50095: AMBARI-17751. Fix the logic of size calculation while dealing with multi-level YARN queue for LLAP config calculations. Also, added validation check for 'hive.server2.tez.session

2016-07-15 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50095/ --- Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. Bugs:

Review Request 50097: HAWQ install should recommend required values for HDFS properties

2016-07-15 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50097/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, Lav Jain, and

Re: Review Request 50091: EU/RU Kafka should come right after ZK instead of before Atlas, fix role command order

2016-07-15 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50091/#review142464 ---

Review Request 50091: EU/RU Kafka should come right after ZK instead of before Atlas, fix role command order

2016-07-15 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50091/ --- Review request for Ambari, Jonathan Hurley, Nate Cole, and Sumit Mohanty.

Re: Review Request 50086: Slider range for should be dynamic on HAWQ settings page

2016-07-15 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50086/#review142449 --- Ship it! Ship It! - Alexander Denissov On July 15, 2016,

Review Request 50087: Ambari should have a script to add new repository and service to existing stack

2016-07-15 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50087/ --- Review request for Ambari, Alexander Denissov, Jayush Luniya, Matt, and Tim

Review Request 50086: Slider range for should be dynamic on HAWQ settings page

2016-07-15 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50086/ --- Review request for Ambari, Alexander Denissov, Jayush Luniya, Matt, and Tim

Re: Review Request 50083: Storm service check failed during Express Upgrade due to Nimbus going down after restart with CNF IClusterReporter

2016-07-15 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50083/#review142445 --- Ship it! Ship It! - Alejandro Fernandez On July 15, 2016,

Re: Review Request 50083: Storm service check failed during Express Upgrade due to Nimbus going down after restart with CNF IClusterReporter

2016-07-15 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50083/#review142444 --- Ship it!

Review Request 50083: Storm service check failed during Express Upgrade due to Nimbus going down after restart with CNF IClusterReporter

2016-07-15 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50083/ --- Review request for Ambari, Alejandro Fernandez and Jonathan Hurley. Bugs:

Re: Review Request 50069: Docker based dev environment for Log Search

2016-07-15 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50069/ --- (Updated July 15, 2016, 8:37 p.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 50079: EclipseLink Sequence Query Stuck Inside of Transaction And Blocks Other Threads

2016-07-15 Thread Sid Wagle
> On July 15, 2016, 5:03 p.m., Sid Wagle wrote: > > Ship It! > > Sid Wagle wrote: > General question: Any reason why we started to see this now? Is it > possible the postgres verion 9.2 does not suffer from this? We seem to be > still installing postgresql-server-8.4.20-6 > > Jonathan

Re: Review Request 50069: Docker based dev environment for Log Search

2016-07-15 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50069/ --- (Updated July 15, 2016, 8:28 p.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 50059: ArrayIndexOutOfBounds exception thrown on shard calculation

2016-07-15 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50059/#review142439 --- Ship it! Ship It! - Sumit Mohanty On July 15, 2016, 5:37

Re: Review Request 50079: EclipseLink Sequence Query Stuck Inside of Transaction And Blocks Other Threads

2016-07-15 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50079/ --- (Updated July 15, 2016, 4:06 p.m.) Review request for Ambari, Alejandro

Re: Review Request 50059: ArrayIndexOutOfBounds exception thrown on shard calculation

2016-07-15 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50059/#review142437 --- Ship it! Ship It! - Aravindan Vijayan On July 15, 2016,

Re: Review Request 50069: Docker based dev environment for Log Search

2016-07-15 Thread Oliver Szabo
> On July 15, 2016, 7:08 p.m., Nate Cole wrote: > > ambari-logsearch/docker/test-logs/zookeeper/zookeeper-test-log.log, lines > > 1-3 > > > > > > Checking in a log file? That is a test file for log processing. I

Re: Review Request 50069: Docker based dev environment for Log Search

2016-07-15 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50069/#review142389 ---

Re: Review Request 49902: Storm nimbus server fails to come up with CNF backtype.storm.metric.IClusterReporter error

2016-07-15 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49902/#review142428 ---

Re: Review Request 49838: AMBARI-17636 Service Configs page: can't see all config versions in dropdown

2016-07-15 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49838/#review142419 --- Ship it! Ship It! - Zhe (Joe) Wang On July 15, 2016, 1:55

Re: Review Request 49987: AMBARI-17687 [RU/Downgrade] ranger service goes down in between of RU downgrade

2016-07-15 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49987/#review142413 --- Ship it! Ship It! - Jayush Luniya On July 14, 2016, 2:23

Re: Review Request 48651: Add unit tests for Spark2 service definition

2016-07-15 Thread Jayush Luniya
> On June 17, 2016, 3:04 p.m., Jayush Luniya wrote: > > @Saisai > > Looks like you need to rebase again > > $ git apply ~/Downloads/AMBARI-16864-v1.patch > > /Users/jluniya/Downloads/AMBARI-16864-v1.patch:1100: trailing whitespace. > > "spark-javaopts-properties": {}, > >

Re: Review Request 48999: Fix Spark2 history server quick link still point to spark1 url

2016-07-15 Thread Jayush Luniya
> On June 21, 2016, 3:22 a.m., Jayush Luniya wrote: > > Ship It! Saisai, Can you close this review? - Jayush --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48999/#review138755

Re: Review Request 50030: AMBARI-17688 Ranger stack changes in Ambari to support secure Solr

2016-07-15 Thread Jayush Luniya
> On July 15, 2016, 5:25 p.m., Jayush Luniya wrote: > > Ship It! Committed patch. Please close review. - Jayush --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50030/#review142401

Re: Review Request 50030: AMBARI-17688 Ranger stack changes in Ambari to support secure Solr

2016-07-15 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50030/#review142401 --- Ship it! Ship It! - Jayush Luniya On July 15, 2016, 8:43

Re: Review Request 50079: EclipseLink Sequence Query Stuck Inside of Transaction And Blocks Other Threads

2016-07-15 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50079/#review142398 --- Ship it! Ship It! - Alejandro Fernandez On July 15, 2016,

Re: Review Request 50079: EclipseLink Sequence Query Stuck Inside of Transaction And Blocks Other Threads

2016-07-15 Thread Sid Wagle
> On July 15, 2016, 5:03 p.m., Sid Wagle wrote: > > Ship It! General question: Any reason why we started to see this now? Is it possible the postgres verion 9.2 does not suffer from this? We seem to be still installing postgresql-server-8.4.20-6 - Sid

Re: Review Request 50079: EclipseLink Sequence Query Stuck Inside of Transaction And Blocks Other Threads

2016-07-15 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50079/#review142395 --- Ship it! Ship It! - Sid Wagle On July 15, 2016, 3:56 p.m.,

Re: Review Request 50079: EclipseLink Sequence Query Stuck Inside of Transaction And Blocks Other Threads

2016-07-15 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50079/#review142388 --- Ship it! Ship It! - Nate Cole On July 15, 2016, 11:56 a.m.,

Review Request 50079: EclipseLink Sequence Query Stuck Inside of Transaction And Blocks Other Threads

2016-07-15 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50079/ --- Review request for Ambari, Alejandro Fernandez, Nate Cole, and Sid Wagle.

Re: Review Request 49995: AMBARI-17695 : AMS Split point calculation not optimal for large clusters.

2016-07-15 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49995/#review142382 --- Ship it! Ship It! - Dmytro Sen On Июль 14, 2016, 8:26 п.п.,

Re: Review Request 49387: Support password type for custom properties

2016-07-15 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49387/#review142375 ---

Re: Review Request 49891: Restart indicator is not shown after modifying the configs

2016-07-15 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49891/#review142373 --- Ship it!

Re: Review Request 50076: Zeppelin server should have yarn-client installed

2016-07-15 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50076/#review142374 --- Ship it! Ship It! - Sumit Mohanty On July 15, 2016, 1:26

Review Request 50076: Zeppelin server should have yarn-client installed

2016-07-15 Thread Renjith Kamath
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50076/ --- Review request for Ambari, Alejandro Fernandez, DIPAYAN BHOWMICK, Gaurav Nagar,

Re: Review Request 48972: AMBARI-17253 Ambari Alert causes too many wanings in ZooKeeper logs.

2016-07-15 Thread Nate Cole
> On July 12, 2016, 8:30 p.m., Jonathan Hurley wrote: > > I'm going to +1 this, but I'd still like to know how other commiters feel > > about making the socket commands paramters vs actual parts of the source > > element. > > Jayush Luniya wrote: > Parameters make more sense for script

Review Request 50074: Tez views disappear and can not be created after upgrade to 2.4.0.0

2016-07-15 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50074/ --- Review request for Ambari, DIPAYAN BHOWMICK, Nitiraj Rathore, Pallav

Re: Review Request 50069: Docker based dev environment for Log Search

2016-07-15 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50069/ --- (Updated July 15, 2016, 10:51 a.m.) Review request for Ambari, Dharmesh

Re: Review Request 49145: Ability to add "javax.jdo.option.ConnectionPassword" to hive clients from ambari

2016-07-15 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49145/#review142362 --- Ship it! Ship It! - Robert Levas On July 14, 2016, 6:57

Re: Review Request 50070: Capacity Scheduler View - Fix for nodelabel api compatibility issue in HDP-2.5

2016-07-15 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50070/#review142361 --- Ship it! Ship It! - Sreenath Somarajapuram On July 15,

Review Request 50070: Capacity Scheduler View - Fix for nodelabel api compatibility issue in HDP-2.5

2016-07-15 Thread Akhil PB
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50070/ --- Review request for Ambari, DIPAYAN BHOWMICK, Pallav Kulshreshtha, and Sreenath

Re: Review Request 49891: Restart indicator is not shown after modifying the configs

2016-07-15 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49891/ --- (Updated July 15, 2016, 1:26 p.m.) Review request for Ambari, Andrew Onischuk,

Review Request 50069: Docker based dev environment for Log Search

2016-07-15 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50069/ --- Review request for Ambari, Dharmesh Makwana, Miklos Gergely, Robert Nettleton,

Re: Review Request 50030: AMBARI-17688 Ranger stack changes in Ambari to support secure Solr

2016-07-15 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50030/#review142357 --- Ship it! Ship It! - Robert Levas On July 15, 2016, 4:43

Re: Review Request 50068: All devdeploys fail due to Storm components fail to install

2016-07-15 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50068/#review142355 --- Ship it! Ship It! - Dmitro Lisnichenko On July 15, 2016,

Review Request 50068: All devdeploys fail due to Storm components fail to install

2016-07-15 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50068/ --- Review request for Ambari and Dmitro Lisnichenko. Bugs: AMBARI-17729

Re: Review Request 49640: Identify config changes added to Ambari-2.4.0 and mark them to not get added during Ambari upgrade

2016-07-15 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49640/#review142354 --- Ship it! Ship It! - Andrew Onischuk On July 15, 2016, 9:41

Review Request 50067: Error message does not deliver when executing ambari-server command as a non-root user

2016-07-15 Thread wang yaoxin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50067/ --- Review request for Ambari. Bugs: AMBARI-17728

Re: Review Request 49640: Identify config changes added to Ambari-2.4.0 and mark them to not get added during Ambari upgrade

2016-07-15 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49640/ --- (Updated July 15, 2016, 12:41 p.m.) Review request for Ambari, Andrew

Re: Review Request 50030: AMBARI-17688 Ranger stack changes in Ambari to support secure Solr

2016-07-15 Thread Mugdha Varadkar
> On July 14, 2016, 6:30 p.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py, > > line 630 > > > > > > Whynot use `hive_server2_keytab`

Re: Review Request 50030: AMBARI-17688 Ranger stack changes in Ambari to support secure Solr

2016-07-15 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50030/ --- (Updated July 15, 2016, 8:43 a.m.) Review request for Ambari, Alejandro

Re: Review Request 49755: Capacity Scheduler View - Fetching current RM configuration of queues and preemption implementation

2016-07-15 Thread Akhil PB
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49755/ --- (Updated July 15, 2016, 7:29 a.m.) Review request for Ambari, DIPAYAN