Review Request 64554: HBase Cannot Find LZO Classes After Being Patched

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

Review Request 64553: Log Feeder refactor: integrate with spring boot

2017-12-12 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64553/ --- Review request for Ambari, Krisztian Kasa and Sid Wagle. Bugs: AMBARI-22639

Re: Review Request 64513: Update the default storm log search configuration (use wildcards)

2017-12-12 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64513/#review193574 --- Ship it! Ship It! - Robert Nettleton On Dec. 12, 2017, 4:24

Re: Review Request 64118: Refactor internal code of handling info between kerberos wizard actions

2017-12-12 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64118/#review193576 --- This is looking really good. Upgrade logic to the appropriate

Re: Review Request 64544: Migrate user data for upgrade to improved user account management

2017-12-12 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64544/#review193572 --- Ship it! Ship It! - Attila Magyar On Dec. 12, 2017, 2:35

Re: Review Request 64537: AMBARI-22635 : Ambari should create a dummy core-site.xml for Ranger plugins when namenode is not installed.

2017-12-12 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64537/#review193559 --- Instead of putting empty files on the classpath isn't it easier

Re: Review Request 64544: Migrate user data for upgrade to improved user account management

2017-12-12 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64544/#review193564 ---

Re: Review Request 64407: AMBARI-22592: Set ambari.repo. in ambari-server setup script

2017-12-12 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64407/#review193562 --- ambari-server/src/test/python/TestAmbariServer.py Lines 3703

Re: Review Request 64528: Zeppelin Interpreter settings are getting updated after zeppelin restart

2017-12-12 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64528/#review193570 --- Ship it! Ship It! - Jayush Luniya On Dec. 12, 2017, 4:21

Re: Review Request 64537: AMBARI-22635 : Ambari should create a dummy core-site.xml for Ranger plugins when namenode is not installed.

2017-12-12 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64537/#review193566 --- Seems like lots of code duplication, can you centralize this in a

Review Request 64552: Fix misuses of os.path.dirname(path) in yarn.py

2017-12-12 Thread Akira Ajisaka
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64552/ --- Review request for Ambari, Andrew Onischuk and Dmytro Sen. Bugs: AMBARI-22637

Review Request 64568: AMBARI-22638. yarn.timeline-service property enabled overriding preset secure blueprint config for the same

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

Review Request 64537: AMBARI-22635 : Ambari should create a dummy core-site.xml for Ranger plugins when namenode is not installed.

2017-12-12 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64537/ --- Review request for Ambari, Gautam Borad, Jonathan Hurley, Jayush Luniya,

Review Request 64531: AMBARI-22633 MapDate provides the date incorrectly when Filter is cloned and used in multi-threads

2017-12-12 Thread Jungtaek Lim
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64531/ --- Review request for Ambari, Miklos Gergely, Oliver Szabo, and Sid Wagle. Bugs:

Review Request 64544: Migrate user data for upgrade to improved user account management

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

Review Request 64542: Kerberos support for OneFS

2017-12-12 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64542/ --- Review request for Ambari, Eugene Chekanskiy, Nate Cole, Robert Levas, and

Re: Review Request 64528: Zeppelin Interpreter settings are getting updated after zeppelin restart

2017-12-12 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64528/#review193521 --- Ship it! Ship It! - Gaurav Nagar On Dec. 12, 2017, 7:39

Re: Review Request 64478: Blueprint V2 API: object model, resource provider, cluster creation

2017-12-12 Thread Balázs Bence Sári
> On Dec. 11, 2017, 10:23 p.m., Madhuvanthi Radhakrishnan wrote: > > ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql > > Line 42 (original) > > > > > > Where are we setting this value? I saw we removed

Re: Review Request 64531: AMBARI-22633 MapDate provides the date incorrectly when Filter is cloned and used in multi-threads

2017-12-12 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64531/#review193537 --- Ship it! Pobably that would be better if we would clone all of

Re: Review Request 64478: Blueprint V2 API: object model, resource provider, cluster creation

2017-12-12 Thread Balázs Bence Sári
> On Dec. 11, 2017, 10:23 p.m., Madhuvanthi Radhakrishnan wrote: > > ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql > > Line 42 (original) > > > > > > Where are we setting this value? I saw we removed

Review Request 64547: Hive MetaStore does not start even after passing the mysql-connector to ambari-server

2017-12-12 Thread Myroslav Papirkovskyy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64547/ --- Review request for Ambari, Andrew Onischuk and Sid Wagle. Bugs: AMBARI-22631

Re: Review Request 64547: Hive MetaStore does not start even after passing the mysql-connector to ambari-server

2017-12-12 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64547/#review193546 --- Ship it! Ship It! - Andrew Onischuk On Dec. 12, 2017, 3:49

Re: Review Request 64242: AMBARI-22476: Modify bootstrap API to pass Ambari repo URL from UI and retrieve Host OS types

2017-12-12 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64242/#review193548 --- Ship it! Will mark this as Ship It, but the repo url should be

Re: Review Request 64407: AMBARI-22592: Set ambari.repo. in ambari-server setup script

2017-12-12 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64407/#review193547 --- Ship it! Ship It! - Nate Cole On Dec. 8, 2017, 5:57 a.m.,

Re: Review Request 64242: AMBARI-22476: Modify bootstrap API to pass Ambari repo URL from UI and retrieve Host OS types

2017-12-12 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64242/#review193550 --- This has been pushed. Please close this review and associated

Re: Review Request 64407: AMBARI-22592: Set ambari.repo. in ambari-server setup script

2017-12-12 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64407/#review193549 --- This has been pushed. Please close this review and associated

Re: Review Request 64542: Kerberos support for OneFS

2017-12-12 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64542/#review193551 --- Fix it, then Ship it!

Re: Review Request 64544: Migrate user data for upgrade to improved user account management

2017-12-12 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64544/#review193553 --- Ship it! Ship It! - Nate Cole On Dec. 12, 2017, 9:35 a.m.,

Re: Review Request 64528: Zeppelin Interpreter settings are getting updated after zeppelin restart

2017-12-12 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64528/#review193554 --- Ship it! Ship It! - Sid Wagle On Dec. 12, 2017, 7:39 a.m.,

Re: Review Request 64513: Update the default storm log search configuration (use wildcards)

2017-12-12 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64513/#review193555 --- Ship it! Ship It! - Sid Wagle On Dec. 11, 2017, 11:47 p.m.,

Re: Review Request 64513: Update the default storm log search configuration (use wildcards)

2017-12-12 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64513/ --- (Updated Dec. 12, 2017, 4:19 p.m.) Review request for Ambari, Krisztian Kasa,

Re: Review Request 64528: Zeppelin Interpreter settings are getting updated after zeppelin restart

2017-12-12 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64528/ --- (Updated Dec. 12, 2017, 9:51 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 64513: Update the default storm log search configuration (use wildcards)

2017-12-12 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64513/#review193556 --- Last revision has noly whitespace changes? - Sid Wagle On Dec.

Re: Review Request 64513: Update the default storm log search configuration (use wildcards)

2017-12-12 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64513/ --- (Updated Dec. 12, 2017, 4:24 p.m.) Review request for Ambari, Krisztian Kasa,