Re: Review Request 52264: AMBARI-18466 Component should be renamed to "Microsoft R Server Client"

2016-09-27 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52264/#review150529 --- Ship it! Ship It! - Sebastian Toader On Sept. 26, 2016, 8:07

Re: Review Request 52266: Ambari server does not come up after restart if cluster install fails

2016-09-27 Thread Balázs Bence Sári
> On Sept. 26, 2016, 9:33 p.m., Sid Wagle wrote: > > ambari-server/src/main/java/org/apache/ambari/server/stack/UpdateActiveRepoVersionOnStartup.java, > > line 97 > > > > > > When I tried this on my cluster that was

Re: Review Request 52264: AMBARI-18466 Component should be renamed to "Microsoft R Server Client"

2016-09-27 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52264/#review150532 --- Ship it! Ship It! - Balázs Bence Sári On Sept. 26, 2016, 6:0

Review Request 52298: Refactor yarn() function in YARN service. Part 1.

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

Re: Review Request 52298: Refactor yarn() function in YARN service. Part 1.

2016-09-27 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52298/ --- (Updated Sept. 27, 2016, 9:13 a.m.) Review request for Ambari, Jayush Luniya an

Re: Review Request 52281: Fix custom jdbc functionality for ambari-server setup

2016-09-27 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52281/#review150536 --- Ship it! Ship It! - Dmytro Sen On Сен. 26, 2016, 9:40 п.п.,

Re: Review Request 52298: Refactor yarn() function in YARN service. Part 1.

2016-09-27 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52298/ --- (Updated Sept. 27, 2016, 10:05 a.m.) Review request for Ambari, Jayush Luniya a

Re: Review Request 52260: AMBARI-18459: Print error messages if bulkcommand section of a component includes non-existing component

2016-09-27 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52260/#review150542 --- Ship it! Ship It! - Tim Thorpe On Sept. 26, 2016, 2:26 p.m.,

Re: Review Request 52155: ambari server upgrade ambari to 2.1.1 duplicate key error

2016-09-27 Thread wang yaoxin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52155/ --- (Updated 九月 27, 2016, 12:03 p.m.) Review request for Ambari, Jonathan Hurley, O

Re: Review Request 52155: ambari server upgrade ambari to 2.1.1 duplicate key error

2016-09-27 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52155/#review150545 --- ambari-server/src/main/java/org/apache/ambari/server/upgrade/Upgr

Re: Review Request 52282: RU/EU cannot start because ServiceCheckValidityCheck incorrectly calculates Service Checks that ran

2016-09-27 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52282/#review150546 --- Ship it! Ship It! - Jonathan Hurley On Sept. 26, 2016, 5:53

Re: Review Request 52289: Regression: krb5JAASLogin.conf is not updated during secure BP install

2016-09-27 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52289/#review150547 --- Ship it! Ship It! - Jonathan Hurley On Sept. 26, 2016, 8:09

Re: Review Request 52289: Regression: krb5JAASLogin.conf is not updated during secure BP install

2016-09-27 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52289/#review150548 --- Ship it! Ship It! - Sandor Magyari On Sept. 27, 2016, 12:09

Re: Review Request 52282: RU/EU cannot start because ServiceCheckValidityCheck incorrectly calculates Service Checks that ran

2016-09-27 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52282/#review150558 --- Ship it! Ship It! - Nate Cole On Sept. 26, 2016, 5:53 p.m.,

Re: Review Request 52289: Regression: krb5JAASLogin.conf is not updated during secure BP install

2016-09-27 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52289/#review150559 --- Ship it! Ship It! - Nate Cole On Sept. 26, 2016, 8:09 p.m.,

Re: Review Request 52289: Regression: krb5JAASLogin.conf is not updated during secure BP install

2016-09-27 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52289/#review150561 --- Ship it! Ship It! - Robert Nettleton On Sept. 27, 2016, 12:0

Re: Review Request 52281: Fix custom jdbc functionality for ambari-server setup

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

Re: Review Request 52281: Fix custom jdbc functionality for ambari-server setup

2016-09-27 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52281/#review150562 --- Ship it! Ship It! - Dmytro Sen On Сен. 27, 2016, 2:50 п.п.,

Re: Review Request 52223: AMBARI-18051 - Services should be able to provide their own pre-req checks by supplying a jar file

2016-09-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52223/#review150563 --- ambari-server/src/test/java/org/apache/ambari/server/sample/check

Re: Review Request 52298: Refactor yarn() function in YARN service. Part 1.

2016-09-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52298/#review150572 --- Ship it! Ship It! - Alejandro Fernandez On Sept. 27, 2016, 1

Review Request 52304: Scope and Services need to be used for orchestration

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

Re: Review Request 52266: Ambari server does not come up after restart if cluster install fails

2016-09-27 Thread Sid Wagle
> On Sept. 26, 2016, 9:33 p.m., Sid Wagle wrote: > > ambari-server/src/main/java/org/apache/ambari/server/stack/UpdateActiveRepoVersionOnStartup.java, > > line 97 > > > > > > When I tried this on my cluster that was

Re: Review Request 52304: Scope and Services need to be used for orchestration

2016-09-27 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52304/#review150584 --- Ship it! ambari-server/src/main/java/org/apache/ambari/server/

Re: Review Request 52304: Scope and Services need to be used for orchestration

2016-09-27 Thread Nate Cole
> On Sept. 27, 2016, 2:13 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java, > > lines 825-828 > > > > > > Is this necessarily tru

Re: Review Request 51814: Update sinks to read multiple collector hostnames from configs

2016-09-27 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51814/#review150614 --- ambari-metrics/ambari-metrics-common/src/main/java/org/apache/had

Review Request 52335: Alert unit tests failing on trunk

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

Re: Review Request 52335: Alert unit tests failing on trunk

2016-09-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52335/#review150683 --- Ship it! Ship It! - Sumit Mohanty On Sept. 28, 2016, 5:10 a.

Re: Review Request 51647: AMBARI-18313: Capacity Scheduler View: Xml diff view tool to show changes made and queue capacity chart

2016-09-27 Thread Sreenath Somarajapuram
> On Sept. 23, 2016, 10:19 a.m., Sreenath Somarajapuram wrote: > > contrib/views/capacity-scheduler/src/main/resources/ui/bower.json, line 16 > > > > > > Package versiosn must be fixed. > > Akhil PB wrote: > Blob