Re: Review Request 64046: Incorrect pie chart distribution

2017-11-22 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64046/#review191785 --- Ship it! Ship It! - venkat sairam On Nov. 23, 2017, 7:04

Review Request 64046: Incorrect pie chart distribution

2017-11-22 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64046/ --- Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, Rohit

Re: Review Request 64035: Workflow Manager View - FS node will overwrite internal commands and replace them with blank "move" commands when reopening the node

2017-11-22 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64035/#review191783 --- Ship it! Ship It! - Prabhjyot Singh On Nov. 22, 2017, 10:54

Re: Review Request 64035: Workflow Manager View - FS node will overwrite internal commands and replace them with blank "move" commands when reopening the node

2017-11-22 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64035/#review191782 --- Ship it! Ship It! - Padma Priya N On Nov. 22, 2017, 5:24

Re: Review Request 64035: Workflow Manager View - FS node will overwrite internal commands and replace them with blank "move" commands when reopening the node

2017-11-22 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64035/#review191778 --- Ship it! Ship It! - Pallav Kulshreshtha On Nov. 22, 2017,

Re: Review Request 64005: Fix build for test classes in feature branch

2017-11-22 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64005/#review191774 --- Ship it! Ship It! - Swapan Shridhar On Nov. 22, 2017, 3:01

Review Request 64043: SNMP Trap should contain sysuptime field filled

2017-11-22 Thread Amarnath reddy pappu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64043/ --- Review request for Ambari, Jonathan Hurley and Nate Cole. Bugs: AMBARI-22458

Re: Review Request 64020: AMBARI-22496. Admin View - Manage Versions page should show HDP-GPL repo if necessary

2017-11-22 Thread Denys Buzhor
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64020/#review191770 --- Ship it! Ship It! - Denys Buzhor On Nov. 23, 2017, 12:43

Re: Review Request 64020: AMBARI-22496. Admin View - Manage Versions page should show HDP-GPL repo if necessary

2017-11-22 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64020/ --- (Updated Nov. 23, 2017, 12:43 a.m.) Review request for Ambari, Alexandr

Re: Review Request 64020: AMBARI-22496. Admin View - Manage Versions page should show HDP-GPL repo if necessary

2017-11-22 Thread Denys Buzhor
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64020/#review191764 --- Ship it! Ship It! - Denys Buzhor On Nov. 22, 2017, 3:42

Re: Review Request 64017: Installer - Select Version page should show HDP-GPL repo if necessary

2017-11-22 Thread Denys Buzhor
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64017/#review191763 --- Ship it! Ship It! - Denys Buzhor On Nov. 22, 2017, 2:25

Re: Review Request 63957: Stack advisor should disallow lzo enable without accepting license agreement.

2017-11-22 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63957/#review191756 --- Ship it! Ship It! - Nate Cole On Nov. 22, 2017, 1:09 p.m.,

Re: Review Request 63957: Stack advisor should disallow lzo enable without accepting license agreement.

2017-11-22 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63957/#review191752 --- Ship it! Ship It! - Sid Wagle On Nov. 22, 2017, 6:09 p.m.,

Re: Review Request 64024: AMBARI-22498. Remove trailing lines (if any) from llapstatus command before converting the o/p to JSON.

2017-11-22 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64024/#review191751 --- Ship it! Ship It! - Madhuvanthi Radhakrishnan On Nov. 22,

Re: Review Request 63957: Stack advisor should disallow lzo enable without accepting license agreement.

2017-11-22 Thread Myroslav Papirkovskyy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63957/ --- (Updated Лис. 22, 2017, 8:09 після полудня) Review request for Ambari, Andrew

Re: Review Request 64036: Log Search UI: refine time range picker

2017-11-22 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64036/#review191749 --- Ship it! Ship It! - Aleksandr Kovalenko On Ноя. 22, 2017,

Review Request 64036: Log Search UI: refine time range picker

2017-11-22 Thread Andrii Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64036/ --- Review request for Ambari and Aleksandr Kovalenko. Bugs: AMBARI-22503

Re: Review Request 63965: New endpoint service_config_version for creating new config versions via service endpoint instead of cluster update

2017-11-22 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63965/ --- (Updated Nov. 22, 2017, 5:47 p.m.) Review request for Ambari, Jayush Luniya,

Review Request 64035: Workflow Manager View - FS node will overwrite internal commands and replace them with blank "move" commands when reopening the node

2017-11-22 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64035/ --- Review request for Ambari, Gaurav Nagar, Padma Priya N, Pallav Kulshreshtha,

Re: Review Request 63937: AMBARI-22472. Ambari Upgrade 2.5 -> 2.6 : Update NodeManager's HSI identity 'llap_zk_hive' and 'llap_task_hive' to use '/HIVE/HIVE_SERVER/hive_server_hive' reference instead

2017-11-22 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63937/#review191745 --- Ship it! Ship It! - Madhuvanthi Radhakrishnan On Nov. 22,

Re: Review Request 64030: Modify AMBARI-22387 to Check for LZO + No Opt-in

2017-11-22 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64030/ --- (Updated Nov. 22, 2017, 5:43 p.m.) Review request for Ambari, Jonathan Hurley

Re: Review Request 64030: Modify AMBARI-22387 to Check for LZO + No Opt-in

2017-11-22 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64030/#review191741 --- Ship it! Ship It! - Nate Cole On Nov. 22, 2017, 9 a.m.,

Re: Review Request 64005: Fix build for test classes in feature branch

2017-11-22 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64005/ --- (Updated Лис. 22, 2017, 2:59 після полудня) Review request for Ambari, Jayush

Re: Review Request 63961: Ambari upgrade failed

2017-11-22 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63961/#review191735 --- Ship it! Ship It! - Jonathan Hurley On Nov. 21, 2017, 12:56

Re: Review Request 64030: Modify AMBARI-22387 to Check for LZO + No Opt-in

2017-11-22 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64030/#review191734 --- Ship it! Ship It! - Jonathan Hurley On Nov. 22, 2017, 9

Re: Review Request 64027: Ambari server becomes unusable when config properties are misconfigured

2017-11-22 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64027/#review191733 --- Ship it! Ship It! - Attila Magyar On Nov. 22, 2017, 12:52

Review Request 64030: Modify AMBARI-22387 to Check for LZO + No Opt-in

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

Re: Review Request 64027: Ambari server becomes unusable when config properties are misconfigured

2017-11-22 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64027/#review191729 --- Ship it! Ship It! - Miklos Gergely On Nov. 22, 2017, 12:52

Re: Review Request 64027: Ambari server becomes unusable when config properties are misconfigured

2017-11-22 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64027/ --- (Updated Nov. 22, 2017, 1:52 p.m.) Review request for Ambari, Attila Magyar,

Review Request 64027: Ambari server becomes unusable when config properties are misconfigured

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

Re: Review Request 63965: New endpoint service_config_version for creating new config versions via service endpoint instead of cluster update

2017-11-22 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63965/#review191593 --- Fix it, then Ship it!

Review Request 64024: AMBARI-22498. Remove trailing lines (if any) from llapstatus command before converting the o/p to JSON.

2017-11-22 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64024/ --- Review request for Ambari, Jayush Luniya and Madhuvanthi Radhakrishnan. Bugs: