Re: Review Request 59948: AMBARI-20853. Service Advisor - Allow Service to define its Advisor Type as Python or Java

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

Re: Review Request 59948: AMBARI-20853. Service Advisor - Allow Service to define its Advisor Type as Python or Java

2017-06-09 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59948/#review177531 --- ambari-logsearch/ambari-logsearch-it/pom.xml Line 84 (original),

Re: Review Request 59948: AMBARI-20853. Service Advisor - Allow Service to define its Advisor Type as Python or Java

2017-06-09 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59948/ --- (Updated June 10, 2017, 12:28 a.m.) Review request for Ambari, Jonathan

Re: Review Request 59948: AMBARI-20853. Service Advisor - Allow Service to define its Advisor Type as Python or Java

2017-06-09 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59948/ --- (Updated June 10, 2017, 12:28 a.m.) Review request for Ambari, Jonathan

Re: Review Request 59951: AMBARI-21214 : Use a uuid vs long row key for metrics in AMS schema

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

Re: Review Request 59956: Update Database Access Layer to Support New Database Schema for Improved User Account Management

2017-06-09 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59956/#review177516 ---

Review Request 59956: Update Database Access Layer to Support New Database Schema for Improved User Account Management

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

Re: Review Request 59955: AMBARI-21218 Enable Kerberos UI is blank the first time opens it

2017-06-09 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59955/#review177515 --- Ship it! Ship It! - Tim Thorpe On June 9, 2017, 8:07 p.m.,

Review Request 59955: AMBARI-21218 Enable Kerberos UI is blank the first time opens it

2017-06-09 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59955/ --- Review request for Ambari and Tim Thorpe. Bugs: AMBARI-21218

Re: Review Request 59953: AMBARI-21213 Consolidate UI for the "Loading..." text

2017-06-09 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59953/#review177507 --- Ship it! Ship It! - Tim Thorpe On June 9, 2017, 7:01 p.m.,

Review Request 59953: AMBARI-21213 Consolidate UI for the "Loading..." text

2017-06-09 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59953/ --- Review request for Ambari and Tim Thorpe. Bugs: AMBARI-21213

Re: Review Request 59951: AMBARI-21214 : Use a uuid vs long row key for metrics in AMS schema

2017-06-09 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59951/ --- (Updated June 9, 2017, 6:52 p.m.) Review request for Ambari, Dmytro Sen, Sumit

Review Request 59951: AMBARI-21214 : Use a uuid vs long row key for metrics in AMS schema

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

Review Request 59948: AMBARI-20853. Service Advisor - Allow Service to define its Advisor Type as Python or Java

2017-06-09 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59948/ --- Review request for Ambari, Jonathan Hurley, Nate Cole, Sumit Mohanty, Swapan

Re: Review Request 59944: Add ability to Log Search to test a log entry if it is parseable

2017-06-09 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59944/#review177498 ---

Re: Review Request 59945: Missing os_family causes a warning during host check

2017-06-09 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59945/#review177497 --- Ship it! Ship It! - Alejandro Fernandez On June 9, 2017,

Re: Review Request 59920: Yarn stopped by itself after start. HA run

2017-06-09 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59920/#review177496 --- Ship it! nice - Alejandro Fernandez On June 9, 2017, 11:59

Re: Review Request 59947: Remove unnecessary Log Search stack definition parts

2017-06-09 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59947/#review177492 --- Ship it! Ship It! - Oliver Szabo On June 9, 2017, 4:20

Review Request 59947: Remove unnecessary Log Search stack definition parts

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

Re: Review Request 59828: Ability to anonymize data during log processing

2017-06-09 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59828/#review177487 --- Ship it! The patch looks fine. One thing that may be worth

Re: Review Request 59943: Incorrect ambari DDL script for Oracle DB

2017-06-09 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59943/#review177474 --- Ship it! Ship It! - Jonathan Hurley On June 9, 2017, 5:53

Re: Review Request 59945: Missing os_family causes a warning during host check

2017-06-09 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59945/#review177473 --- Ship it! Ship It! - Sebastian Toader On June 9, 2017, 3:13

Re: Review Request 59941: Upgrade PreUpgradeCheck NullPointerException

2017-06-09 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59941/#review177471 --- Ship it! Interesting - how did you come across this NPE? Every

Review Request 59945: Missing os_family causes a warning during host check

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

Re: Review Request 59920: Yarn stopped by itself after start. HA run

2017-06-09 Thread Dmytro Sen
> On Июнь 8, 2017, 4:47 п.п., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/resourcemanager.py > > Lines 188 (patched) > > > > > > ZK Client is not

Re: Review Request 59920: Yarn stopped by itself after start. HA run

2017-06-09 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59920/ --- (Updated Июнь 9, 2017, 11:59 д.п.) Review request for Ambari, Sumit Mohanty,

Re: Review Request 57583: HDFS Bytes Written and HDFS Bytes Read maybe not correct

2017-06-09 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57583/#review177463 --- Ship it! Ship It! - Dmytro Sen On Июнь 9, 2017, 9:58 д.п.,

Re: Review Request 57265: null in heapmap page

2017-06-09 Thread zhangxiaolu zhangxiaolu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57265/ --- (Updated 六月 9, 2017, 9:59 a.m.) Review request for Ambari, Aleksandr

Re: Review Request 57583: HDFS Bytes Written and HDFS Bytes Read maybe not correct

2017-06-09 Thread zhangxiaolu zhangxiaolu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57583/ --- (Updated 六月 9, 2017, 9:58 a.m.) Review request for Ambari, Dmytro Sen, Jaimin

Re: Review Request 59943: Incorrect ambari DDL script for Oracle DB

2017-06-09 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59943/#review177455 --- Ship it! Ship It! - Dmytro Sen On Июнь 9, 2017, 9:53 д.п.,

Review Request 59943: Incorrect ambari DDL script for Oracle DB

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