Re: Review Request 59503: Create custom admin user during ambari server setup

2017-05-31 Thread Anita Jebaraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59503/ --- (Updated June 1, 2017, 5:54 a.m.) Review request for Ambari, Alejandro

Review Request 59694: Enabling Kerberos should use cluster stack id

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

Re: Review Request 59693: Upgrade Execution Commands Are Missing Service/Component For Action Commands

2017-05-31 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59693/#review176525 --- Ship it! Ship It! - Nate Cole On May 31, 2017, 4:55 p.m.,

Re: Review Request 59684: Log more info about heartbeat message/response when server - agent communication gets out of sync

2017-05-31 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59684/#review176517 --- Ship it! Ship It! - Attila Doroszlai On May 31, 2017, 3:49

Re: Review Request 59689: Fix swagger sytax errors in Log Search REST API doc

2017-05-31 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59689/#review176516 --- Ship it! Ship It! - Miklos Gergely On May 31, 2017, 6:53

Review Request 59693: Upgrade Execution Commands Are Missing Service/Component For Action Commands

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

Re: Review Request 59689: Fix swagger sytax errors in Log Search REST API doc

2017-05-31 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59689/#review176511 --- Ship it! Ship It! - Robert Nettleton On May 31, 2017, 6:53

Review Request 59689: Fix swagger sytax errors in Log Search REST API doc

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

Re: Review Request 59683: Design Ambari Infra Component & workflows

2017-05-31 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59683/#review176485 --- Ship it! Ship It! - Robert Nettleton On May 31, 2017, 1:51

Re: Review Request 59609: Export cluster_name in the cluster template file

2017-05-31 Thread Sangeeta Ravindran
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59609/#review176474 --- Ship it! Ship It! - Sangeeta Ravindran On May 30, 2017,

Re: Review Request 59613: Roles below Cluster Administrator should not be allowed to edit repositories and install stack versions

2017-05-31 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59613/#review176470 --- Ship it! Ship It! - Alexandr Antonenko On May 26, 2017,

Re: Review Request 59609: Export cluster_name in the cluster template file

2017-05-31 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59609/#review176459 --- Ship it! Ship It! - Di Li On May 30, 2017, 10:08 p.m.,

Re: Review Request 59684: Log more info about heartbeat message/response when server - agent communication gets out of sync

2017-05-31 Thread Myroslav Papirkovskyy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59684/#review176456 --- Ship it! Ship It! - Myroslav Papirkovskyy On Травень 31,

Re: Review Request 59648: Add a flag to indicate NN restart is rolling

2017-05-31 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59648/#review176447 --- Ship it! Ship It! - Dmitro Lisnichenko On May 31, 2017,

Re: Review Request 59648: Add a flag to indicate NN restart is rolling

2017-05-31 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59648/#review176446 --- Ship it! Ship It! - Eugene Chekanskiy On Травень 31, 2017,

Re: Review Request 59648: Add a flag to indicate NN restart is rolling

2017-05-31 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59648/ --- (Updated Травень 31, 2017, 2:14 після полудня) Review request for Ambari,

Re: Review Request 59683: Design Ambari Infra Component & workflows

2017-05-31 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59683/ --- (Updated May 31, 2017, 1:51 p.m.) Review request for Ambari, Miklos Gergely,

Review Request 59684: Log more info about heartbeat message/response when server - agent communication gets out of sync

2017-05-31 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59684/ --- Review request for Ambari, Attila Doroszlai and Myroslav Papirkovskyy. Bugs:

Re: Review Request 59683: Design Ambari Infra Component & workflows

2017-05-31 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59683/#review176444 --- Ship it! Ship It! - Miklos Gergely On May 31, 2017, 10:31

Re: Review Request 59095: AMBARI-20967 - Remove Zookeeper as a required service from HDFS

2017-05-31 Thread Jonathan Hurley
> On May 30, 2017, 5:42 p.m., Jonathan Hurley wrote: > > I think that the overall problem here is how the UI determines that before > > you can enable NN HA, you must first add ZK to the cluster. Alejandro's > > concern is that the UI shouldn't have this responsibility - it should be > >

Re: Review Request 59095: AMBARI-20967 - Remove Zookeeper as a required service from HDFS

2017-05-31 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59095/#review176442 --- Ship it! Ship It! - Jonathan Hurley On May 9, 2017, 10:39

Re: Review Request 59682: AMBARI-19149. Code cleanup: concatenation in debug messages, unnecessary toString calls

2017-05-31 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59682/#review176440 --- Ship it! Ship It! - Jonathan Hurley On May 31, 2017, 7:07

Re: Review Request 59682: AMBARI-19149. Code cleanup: concatenation in debug messages, unnecessary toString calls

2017-05-31 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59682/#review176436 --- Ship it! Ship It! - Laszlo Puskas On May 31, 2017, 11:07

Re: Review Request 59095: AMBARI-20967 - Remove Zookeeper as a required service from HDFS

2017-05-31 Thread Tim Thorpe
> On May 30, 2017, 9:42 p.m., Jonathan Hurley wrote: > > I think that the overall problem here is how the UI determines that before > > you can enable NN HA, you must first add ZK to the cluster. Alejandro's > > concern is that the UI shouldn't have this responsibility - it should be > >

Re: Review Request 59682: AMBARI-19149. Code cleanup: concatenation in debug messages, unnecessary toString calls

2017-05-31 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59682/#review176435 --- Ship it! Ship It! - Sebastian Toader On May 31, 2017, 1:07

Review Request 59682: AMBARI-19149. Code cleanup: concatenation in debug messages, unnecessary toString calls

2017-05-31 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59682/ --- Review request for Ambari, Jonathan Hurley, Laszlo Puskas, Nate Cole, and

Review Request 59683: Design Ambari Infra Component & workflows

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

Re: Review Request 59679: Hdfs directory viewer should be changed from tree view to list view.

2017-05-31 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59679/#review176429 --- Ship it! Ship It! - belliraj hb On May 31, 2017, 6:32 a.m.,

Re: Review Request 59639: Blueprint export should allow tokenized values in SingleHostUpdater

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

Review Request 59679: Hdfs directory viewer should be changed from tree view to list view.

2017-05-31 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59679/ --- Review request for Ambari, belliraj hb, DIPAYAN BHOWMICK, Nitiraj Rathore,