Review Request 56777: AMBARI-20058 - 'Final' is not shown correctly in config diff tool and in older versions

2017-02-16 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56777/ --- Review request for Ambari, Andriy Babiichuk and Oleg Nechiporenko. Bugs:

Re: Review Request 56772: Storm config change results in Consistency check failed

2017-02-16 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56772/#review165892 --- Ship it! Ship It! - Alexandr Antonenko On Feb. 16, 2017,

Review Request 56772: Storm config change results in Consistency check failed

2017-02-16 Thread Denys Buzhor
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56772/ --- Review request for Ambari and Alexandr Antonenko. Bugs: AMBARI-20059

Re: Review Request 56766: Remove Entities Associated With clusterconfigmapping

2017-02-16 Thread Jonathan Hurley
> On Feb. 16, 2017, 4:55 p.m., Sid Wagle wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/DatabaseConsistencyCheckHelper.java, > > line 227 > > > > > > Looks like a NOOP. Should be deleted?

Re: Review Request 56766: Remove Entities Associated With clusterconfigmapping

2017-02-16 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56766/#review165884 ---

Re: Review Request 56765: AMBARI-20056 : FIFO compaction resulting in too many store files on large cluster

2017-02-16 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56765/#review165882 --- Fix it, then Ship it!

Re: Review Request 56678: PixieDust - Decrease Service Check running time under 3 mins

2017-02-16 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56678/ --- (Updated Feb. 16, 2017, 11:16 p.m.) Review request for Ambari, Alejandro

Review Request 56766: Remove Entities Associated With clusterconfigmapping

2017-02-16 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56766/ --- Review request for Ambari, Nate Cole, Robert Levas, Sebastian Toader, and Sid

Review Request 56765: AMBARI-20056 : FIFO compaction resulting in too many store files on large cluster

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

Re: Review Request 56671: Add Solr authorization settings during LogSearch/Atlas/Ranger startup

2017-02-16 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56671/ --- (Updated Feb. 16, 2017, 7:28 p.m.) Review request for Ambari, Miklos Gergely,

Re: Review Request 56710: Relax ranger config DB consistency check rules

2017-02-16 Thread Alejandro Fernandez
> On Feb. 15, 2017, 9:05 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/DatabaseConsistencyCheckHelper.java, > > line 905 > > > > > > We shouldn't be hardcoding

Re: Review Request 56761: Hosts filter component list is not sorted so it is difficult to use

2017-02-16 Thread Andriy Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56761/#review165867 --- Ship it! Ship It! - Andriy Babiichuk On Лют. 16, 2017, 4:14

Review Request 56761: Hosts filter component list is not sorted so it is difficult to use

2017-02-16 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56761/ --- Review request for Ambari and Andriy Babiichuk. Bugs: AMBARI-20051

Re: Review Request 56641: Change Livy recovery folder permission to 0700

2017-02-16 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56641/#review165864 --- Ship it! Ship It! - Jayush Luniya On Feb. 14, 2017, 5:29

Review Request 56759: AMBARI-20047 : Repository config user needs to set customized service user in Ranger when service plugin is enabled.

2017-02-16 Thread Vishal Suvagia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56759/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Mugdha Varadkar,

Re: Review Request 56671: Add Solr authorization settings during LogSearch/Atlas/Ranger startup

2017-02-16 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56671/ --- (Updated Feb. 16, 2017, 3:57 p.m.) Review request for Ambari, Miklos Gergely,

Re: Review Request 56743: LogFeeder Simulator Enhancements for 3000-node cluster testing

2017-02-16 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56743/#review165859 --- Fix it, then Ship it! Overall, I think the patch is fine, but

Re: Review Request 56606: Allow user to view Tez View after executing query

2017-02-16 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56606/#review165860 --- Ship it! Ship It! - Pallav Kulshreshtha On Feb. 16, 2017,

Review Request 56759: AMBARI-20047 : Repository config user needs to set customized service user in Ranger when service plugin is enabled.

2017-02-16 Thread Vishal Suvagia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56759/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Mugdha Varadkar,

Re: Review Request 56710: Relax ranger config DB consistency check rules

2017-02-16 Thread Jayush Luniya
> On Feb. 15, 2017, 9:05 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/DatabaseConsistencyCheckHelper.java, > > line 905 > > > > > > We shouldn't be hardcoding

Re: Review Request 56751: One way SSL fallback logic can cause some agents to be connected with 2-way SSL

2017-02-16 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56751/#review165856 --- Ship it! Ship It! - Vitalyi Brodetskyi On Лют. 16, 2017,

Re: Review Request 56606: Allow user to view Tez View after executing query

2017-02-16 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56606/ --- (Updated Feb. 16, 2017, 2:33 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 56743: LogFeeder Simulator Enhancements for 3000-node cluster testing

2017-02-16 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56743/#review165850 --- Ship it! Ship It! - Oliver Szabo On Feb. 16, 2017, 9:54

Re: Review Request 56756: Hive View 2.0: Visual Explain-The operators are being shown out of order

2017-02-16 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56756/#review165849 --- Ship it! Ship It! - Pallav Kulshreshtha On Feb. 16, 2017,

Review Request 56756: Hive View 2.0: Visual Explain-The operators are being shown out of order

2017-02-16 Thread Abhishek Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56756/ --- Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Nitiraj Rathore,

Review Request 56751: One way SSL fallback logic can cause some agents to be connected with 2-way SSL

2017-02-16 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56751/ --- Review request for Ambari and Dmitro Lisnichenko. Bugs: AMBARI-20049

Re: Review Request 56750: Don't rerender all widgets when on of them is changed

2017-02-16 Thread Denys Buzhor
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56750/#review165841 --- Ship it! Ship It! - Denys Buzhor On Feb. 16, 2017, 1:29

Review Request 56750: Don't rerender all widgets when on of them is changed

2017-02-16 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56750/ --- Review request for Ambari and Denys Buzhor. Bugs: ambari-20043

Re: Review Request 56671: Add Solr authorization settings during LogSearch/Atlas/Ranger startup

2017-02-16 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56671/ --- (Updated Feb. 16, 2017, 1:28 p.m.) Review request for Ambari, Miklos Gergely,

Re: Review Request 56748: More than one version of jetty jars found in Ambari rpm

2017-02-16 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56748/#review165835 --- Ship it! Ship It! - Vitalyi Brodetskyi On Лют. 16, 2017,

Review Request 56748: More than one version of jetty jars found in Ambari rpm

2017-02-16 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56748/ --- Review request for Ambari, Aravindan Vijayan, Sid Wagle, and Vitalyi Brodetskyi.

Re: Review Request 56627: Cluster deployment fails if cluster template has fewer host_groups than that in blueprint

2017-02-16 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56627/#review165833 --- Yes Robert is right, so there's no such validation because in

Re: Review Request 56745: Workflow Manager workflow rendering is broken in designer page

2017-02-16 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56745/#review165831 --- Ship it! Ship It! - Nitiraj Rathore On Feb. 16, 2017, 10:58

Re: Review Request 56746: Custom job.properties are not retained in the workflow designer

2017-02-16 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56746/#review165829 --- Ship it! Ship It! - belliraj hb On Feb. 16, 2017, 11:40

Review Request 56746: Custom job.properties are not retained in the workflow designer

2017-02-16 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56746/ --- Review request for Ambari, belliraj hb, DIPAYAN BHOWMICK, Gaurav Nagar, Nitiraj

Re: Review Request 56710: Relax ranger config DB consistency check rules

2017-02-16 Thread Andrew Onischuk
> On Feb. 15, 2017, 9:05 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/DatabaseConsistencyCheckHelper.java, > > line 905 > > > > > > We shouldn't be hardcoding

Re: Review Request 56710: Relax ranger config DB consistency check rules

2017-02-16 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56710/ --- (Updated Feb. 16, 2017, 11:24 a.m.) Review request for Ambari, Dmitro

Re: Review Request 56745: Workflow Manager workflow rendering is broken in designer page

2017-02-16 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56745/#review165827 --- Ship it! Ship It! - Gaurav Nagar On Feb. 16, 2017, 10:58

Review Request 56745: Workflow Manager workflow rendering is broken in designer page

2017-02-16 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56745/ --- Review request for Ambari, Gaurav Nagar, Nitiraj Rathore, and Pallav

Re: Review Request 56705: Hive server error on view navigation when wire encryption is enabled

2017-02-16 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56705/#review165817 --- Ship it! Ship It! - Gaurav Nagar On Feb. 15, 2017, 1:01

Re: Review Request 56705: Hive server error on view navigation when wire encryption is enabled

2017-02-16 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56705/#review165816 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Feb. 15, 2017, 1:01