Review Request 57022: After disabling security page is not refreshed

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

Re: Review Request 57022: After disabling security page is not refreshed

2017-02-24 Thread Denys Buzhor
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57022/#review11 --- Ship it! Ship It! - Denys Buzhor On Feb. 24, 2017, 8:45 a.m.

Review Request 57024: AMBARI-20161.Workflow Manager-mapred action contains duplicates in editor if user saves and reopens editor

2017-02-24 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57024/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, and Pallav Kulshreshtha.

Re: Review Request 57024: AMBARI-20161.Workflow Manager-mapred action contains duplicates in editor if user saves and reopens editor

2017-02-24 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57024/#review12 --- Ship it! Ship It! - belliraj hb On Feb. 24, 2017, 9:18 a.m.,

Review Request 57025: AMBARI-20164.Kill a workflow jumps to another tab in WFM

2017-02-24 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57025/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, and Pallav Kulshreshtha.

Re: Review Request 57025: AMBARI-20164.Kill a workflow jumps to another tab in WFM

2017-02-24 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57025/#review14 --- Ship it! Ship It! - belliraj hb On Feb. 24, 2017, 9:34 a.m.,

Re: Review Request 56997: Use storm user principal instead of nimbus user principal for ranger audit

2017-02-24 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56997/#review15 --- ambari-server/src/main/resources/common-services/STORM/1.0.1/kerb

Review Request 57026: Cover NameNode HA wizard with unit tests

2017-02-24 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57026/ --- Review request for Ambari and Aleksandr Kovalenko. Bugs: AMBARI-20165 https

Review Request 57027: AMBARI-20142. Job Link URL not working

2017-02-24 Thread Madhan Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57027/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, and Pallav Kulshreshtha.

Re: Review Request 57027: AMBARI-20142. Job Link URL not working

2017-02-24 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57027/#review18 --- Ship it! Ship It! - belliraj hb On Feb. 24, 2017, 10:09 a.m.

Review Request 57028: AMBARI-20166.Workflow Manager. Nominal time should accept variables.

2017-02-24 Thread Padma Priya N
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57028/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, and Pallav Kulshreshtha.

Re: Review Request 57028: AMBARI-20166.Workflow Manager. Nominal time should accept variables.

2017-02-24 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57028/#review19 --- Ship it! Ship It! - belliraj hb On Feb. 24, 2017, 10:18 a.m.

Review Request 57029: AMBARI-20167. importing from workflow from the coordinator setup hangs

2017-02-24 Thread Madhan Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57029/ --- Review request for Ambari, belliraj hb, Gaurav Nagar, and Pallav Kulshreshtha.

Re: Review Request 57029: AMBARI-20167. importing from workflow from the coordinator setup hangs

2017-02-24 Thread belliraj hb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57029/#review166671 --- Ship it! Ship It! - belliraj hb On Feb. 24, 2017, 10:19 a.m.

Re: Review Request 57024: AMBARI-20161.Workflow Manager-mapred action contains duplicates in editor if user saves and reopens editor

2017-02-24 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57024/#review166673 --- Ship it! Ship It! - Gaurav Nagar On Feb. 24, 2017, 9:18 a.m.

Re: Review Request 57025: AMBARI-20164.Kill a workflow jumps to another tab in WFM

2017-02-24 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57025/#review166674 --- Ship it! Ship It! - Gaurav Nagar On Feb. 24, 2017, 9:34 a.m.

Re: Review Request 57028: AMBARI-20166.Workflow Manager. Nominal time should accept variables.

2017-02-24 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57028/#review166675 --- Ship it! Ship It! - Gaurav Nagar On Feb. 24, 2017, 10:18 a.m

Re: Review Request 57027: AMBARI-20142. Job Link URL not working

2017-02-24 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57027/#review166677 --- Ship it! Ship It! - Gaurav Nagar On Feb. 24, 2017, 10:09 a.m

Re: Review Request 57029: AMBARI-20167. importing from workflow from the coordinator setup hangs

2017-02-24 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57029/#review166676 --- Ship it! Ship It! - Gaurav Nagar On Feb. 24, 2017, 10:19 a.m

Re: Review Request 56994: "Set Version on All Hosts" Fails For Hosts Without Any Stack Components

2017-02-24 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56994/#review166678 --- Ship it! Ship It! - Dmitro Lisnichenko On Feb. 23, 2017, 10:

Review Request 57033: Hive 2: Compact Visual Explain Plan is not loading in firefox

2017-02-24 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57033/ --- Review request for Ambari, Abhishek Kumar, DIPAYAN BHOWMICK, Gaurav Nagar, Nitir

Review Request 57034: Handle on UI error on services API while trying to start all services with atleast one component in INSTALL_FAILED state

2017-02-24 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57034/ --- Review request for Ambari and Andrii Tkach. Bugs: ambari-20169 https://issu

Re: Review Request 57034: Handle on UI error on services API while trying to start all services with atleast one component in INSTALL_FAILED state

2017-02-24 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57034/#review166682 --- Ship it! Ship It! - Andrii Tkach On Feb. 24, 2017, 11:31 a.m

Re: Review Request 56997: Use storm user principal instead of nimbus user principal for ranger audit

2017-02-24 Thread Oliver Szabo
> On Feb. 24, 2017, 9:40 a.m., Mugdha Varadkar wrote: > > ambari-server/src/main/resources/common-services/STORM/1.0.1/kerberos.json, > > line 109 > > > > > > Will this property be updated after ambari upgrade to us

Re: Review Request 57019: HiveView2.0: No notifications for successful query which yields no output.

2017-02-24 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57019/#review166684 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Feb. 24, 2017, 7:35

Re: Review Request 56997: Use storm user principal instead of nimbus user principal for ranger audit

2017-02-24 Thread Oliver Szabo
> On Feb. 24, 2017, 9:40 a.m., Mugdha Varadkar wrote: > > ambari-server/src/main/resources/common-services/STORM/1.0.1/kerberos.json, > > line 109 > > > > > > Will this property be updated after ambari upgrade to us

Review Request 57035: ambari-server package size increased from 645MB to 1GB

2017-02-24 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57035/ --- Review request for Ambari, Attila Doroszlai, DIPAYAN BHOWMICK, Gaurav Nagar, Pal

Review Request 57036: Hive View 2.0: Service checks stopped working.

2017-02-24 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57036/ --- Review request for Ambari, Abhishek Kumar, DIPAYAN BHOWMICK, Gaurav Nagar, Nitir

Re: Review Request 57036: Hive View 2.0: Service checks stopped working.

2017-02-24 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57036/#review166686 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Feb. 24, 2017, 12:07

Re: Review Request 57035: ambari-server package size increased from 645MB to 1GB

2017-02-24 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57035/#review166687 --- Ship it! Ship It! - DIPAYAN BHOWMICK On Feb. 24, 2017, 12:06

Re: Review Request 57036: Hive View 2.0: Service checks stopped working.

2017-02-24 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57036/#review166688 --- Ship it! Ship It! - venkat sairam On Feb. 24, 2017, 12:07 p.

Re: Review Request 57033: Hive 2: Compact Visual Explain Plan is not loading in firefox

2017-02-24 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57033/#review166691 --- Ship it! Ship It! - venkat sairam On Feb. 24, 2017, 11:17 a.

Re: Review Request 57033: Hive 2: Compact Visual Explain Plan is not loading in firefox

2017-02-24 Thread venkat sairam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57033/#review166690 --- Ship it! Ship It! - venkat sairam On Feb. 24, 2017, 11:17 a.

Re: Review Request 57026: Cover NameNode HA wizard with unit tests

2017-02-24 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57026/#review166694 --- Ship it! Ship It! - Aleksandr Kovalenko On Фев. 24, 2017, 10

Re: Review Request 56997: Use storm user principal instead of nimbus user principal for ranger audit

2017-02-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56997/#review166695 --- Ship it! Ship It! - Robert Levas On Feb. 23, 2017, 3:49 p.m.

Re: Review Request 56997: Use storm user principal instead of nimbus user principal for ranger audit

2017-02-24 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56997/#review166697 --- Ship it! Ship It! - Mugdha Varadkar On Feb. 23, 2017, 8:49 p

Re: Review Request 56997: Use storm user principal instead of nimbus user principal for ranger audit

2017-02-24 Thread Robert Levas
> On Feb. 24, 2017, 4:40 a.m., Mugdha Varadkar wrote: > > ambari-server/src/main/resources/common-services/STORM/1.0.1/kerberos.json, > > line 109 > > > > > > Will this property be updated after ambari upgrade to us

Re: Review Request 56997: Use storm user principal instead of nimbus user principal for ranger audit

2017-02-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56997/#review166699 --- ambari-server/src/main/resources/common-services/STORM/1.0.1/kerb

Re: Review Request 56997: Use storm user principal instead of nimbus user principal for ranger audit

2017-02-24 Thread Robert Levas
> On Feb. 24, 2017, 8:25 a.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/STORM/1.0.1/kerberos.json, > > line 109 > > > > > > Make sure the user-defined (artifacts/kerberos_descriptor)

Re: Review Request 57035: ambari-server package size increased from 645MB to 1GB

2017-02-24 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57035/#review166696 --- Fix it, then Ship it! contrib/views/files/pom.xml (line 30)

Review Request 57037: Manage Configuration Group Hosts popup is way too small to be useful when it is needed

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

Re: Review Request 57000: EU: Manual Step without any context after "All Service Checks" fail

2017-02-24 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57000/#review166701 --- Ship it! Ship It! - Jonathan Hurley On Feb. 23, 2017, 8:47 p

Re: Review Request 56997: Use storm user principal instead of nimbus user principal for ranger audit

2017-02-24 Thread Oliver Szabo
> On Feb. 24, 2017, 1:25 p.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/STORM/1.0.1/kerberos.json, > > line 109 > > > > > > Make sure the user-defined (artifacts/kerberos_descriptor)

Re: Review Request 57037: Manage Configuration Group Hosts popup is way too small to be useful when it is needed

2017-02-24 Thread Oleg Nechiporenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57037/#review166703 --- Ship it! Ship It! - Oleg Nechiporenko On Feb. 24, 2017, 1:49

Re: Review Request 56987: TopologyRequest/TopologyLogicalRequest/TopologyHostRequest database inconsistency

2017-02-24 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56987/#review166704 --- Ship it! Ship It! - Sandor Magyari On Feb. 23, 2017, 4:39 p.

Re: Review Request 57033: Hive 2: Compact Visual Explain Plan is not loading in firefox

2017-02-24 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57033/#review166705 --- Ship it! Ship It! - Nitiraj Rathore On Feb. 24, 2017, 11:17

Review Request 57038: Atlas client install fails with blueprint

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

Review Request 57039: Restart Kafka broker failed after enabling kerberos with "kinit(v5): No such file or directory while getting initial credentials"

2017-02-24 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57039/ --- Review request for Ambari, Attila Magyar, Laszlo Puskas, Oliver Szabo, Robert Le

Re: Review Request 57039: Restart Kafka broker failed after enabling kerberos with "kinit(v5): No such file or directory while getting initial credentials"

2017-02-24 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57039/#review166706 --- Ship it! Ship It! - Sandor Magyari On Feb. 24, 2017, 2:19 p.

Review Request 57040: Rebalance HDFS operation returns after the command is issued

2017-02-24 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57040/ --- Review request for Ambari, Robert Levas, Sandor Magyari, and Sebastian Toader.

Re: Review Request 57039: Restart Kafka broker failed after enabling kerberos with "kinit(v5): No such file or directory while getting initial credentials"

2017-02-24 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57039/#review166707 --- Ship it! Ship It! - Attila Magyar On Feb. 24, 2017, 2:19 p.m

Re: Review Request 57038: Atlas client install fails with blueprint

2017-02-24 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57038/#review166709 --- Ship it! Ship It! - Dmitro Lisnichenko On Feb. 24, 2017, 4:1

Review Request 57041: DB selected is not persisted while switching between tabs, SavedQuery tab does not show preview, if the query is not executed before saving

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

Re: Review Request 57039: Restart Kafka broker failed after enabling kerberos with "kinit(v5): No such file or directory while getting initial credentials"

2017-02-24 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57039/#review166710 --- Ship it! Ship It! - Sebastian Toader On Feb. 24, 2017, 3:19

Re: Review Request 57035: ambari-server package size increased from 645MB to 1GB

2017-02-24 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57035/ --- (Updated Feb. 24, 2017, 2:47 p.m.) Review request for Ambari, Attila Doroszlai,

Re: Review Request 57019: HiveView2.0: No notifications for successful query which yields no output.

2017-02-24 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57019/ --- (Updated Feb. 24, 2017, 2:54 p.m.) Review request for Ambari, Abhishek Kumar, D

Re: Review Request 57035: ambari-server package size increased from 645MB to 1GB

2017-02-24 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57035/#review166713 --- Ship it! Ship It! - Gaurav Nagar On Feb. 24, 2017, 2:47 p.m.

Re: Review Request 56987: TopologyRequest/TopologyLogicalRequest/TopologyHostRequest database inconsistency

2017-02-24 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56987/#review166712 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 56987: TopologyRequest/TopologyLogicalRequest/TopologyHostRequest database inconsistency

2017-02-24 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56987/ --- (Updated Feb. 24, 2017, 4:25 p.m.) Review request for Ambari, Attila Doroszlai,

Review Request 57042: Service Flume failed to start with "cannot open /grid/0/log/flume/a1.out for reading: Permission denied"

2017-02-24 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57042/ --- Review request for Ambari, Sumit Mohanty, Sid Wagle, and Vitalyi Brodetskyi. Bu

Re: Review Request 57042: Service Flume failed to start with "cannot open /grid/0/log/flume/a1.out for reading: Permission denied"

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

Re: Review Request 56997: Use storm user principal instead of nimbus user principal for ranger audit

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

Review Request 57043: Zeppelin Notebook fails to start without HDFS via Ambari service

2017-02-24 Thread Renjith Kamath
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57043/ --- Review request for Ambari, Attila Doroszlai, Alejandro Fernandez, DIPAYAN BHOWMI

Re: Review Request 57040: Rebalance HDFS operation returns after the command is issued

2017-02-24 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57040/ --- (Updated Feb. 24, 2017, 4:18 p.m.) Review request for Ambari, Robert Levas, San

Re: Review Request 57043: Zeppelin Notebook fails to start without HDFS via Ambari service

2017-02-24 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57043/#review166718 --- Ship it! Ship It! - Prabhjyot Singh On Feb. 24, 2017, 4:11 p

Re: Review Request 57040: Rebalance HDFS operation returns after the command is issued

2017-02-24 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57040/#review166721 --- Ship it! Ship It! - Sebastian Toader On Feb. 24, 2017, 5:18

Re: Review Request 56987: TopologyRequest/TopologyLogicalRequest/TopologyHostRequest database inconsistency

2017-02-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56987/#review166716 --- Ship it! ambari-server/src/main/java/org/apache/ambari/server/

Re: Review Request 57040: Rebalance HDFS operation returns after the command is issued

2017-02-24 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57040/ --- (Updated Feb. 24, 2017, 4:34 p.m.) Review request for Ambari, Robert Levas, San

Re: Review Request 57043: Zeppelin Notebook fails to start without HDFS via Ambari service

2017-02-24 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57043/#review166722 --- Ship it! Ship It! - Attila Doroszlai On Feb. 24, 2017, 5:11

Re: Review Request 56997: Use storm user principal instead of nimbus user principal for ranger audit

2017-02-24 Thread Robert Levas
> On Feb. 24, 2017, 8:25 a.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/STORM/1.0.1/kerberos.json, > > line 109 > > > > > > Make sure the user-defined (artifacts/kerberos_descriptor)

Re: Review Request 56987: TopologyRequest/TopologyLogicalRequest/TopologyHostRequest database inconsistency

2017-02-24 Thread Sebastian Toader
On Feb. 24, 2017, 5:27 p.m., Sebastian Toader wrote: > > Should any unit tests be updated? Existing unit tests passed fine with the change. Is there already a mechanism in Ambari that can be used to test if certain records are persisted within a transaction or not? - Sebastian

Re: Review Request 57040: Rebalance HDFS operation returns after the command is issued

2017-02-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57040/#review166725 --- Ship it! Ship It! - Robert Levas On Feb. 24, 2017, 11:34 a.m

Re: Review Request 56987: TopologyRequest/TopologyLogicalRequest/TopologyHostRequest database inconsistency

2017-02-24 Thread Robert Levas
On Feb. 24, 2017, 11:27 a.m., Sebastian Toader wrote: > > Should any unit tests be updated? > > Sebastian Toader wrote: > Existing unit tests passed fine with the change. Is there already a > mechanism in Ambari that can be used to test if certain records are persisted > within a transacti

Re: Review Request 57040: Rebalance HDFS operation returns after the command is issued

2017-02-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57040/#review166728 --- ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/p

Re: Review Request 57041: DB selected is not persisted while switching between tabs, SavedQuery tab does not show preview, if the query is not executed before saving

2017-02-24 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57041/#review166734 --- Ship it! Ship It! - Gaurav Nagar On Feb. 24, 2017, 2:31 p.m.

Re: Review Request 57039: Restart Kafka broker failed after enabling kerberos with "kinit(v5): No such file or directory while getting initial credentials"

2017-02-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57039/#review166740 --- Fix it, then Ship it! ambari-server/src/main/resources/common-

Re: Review Request 57039: Restart Kafka broker failed after enabling kerberos with "kinit(v5): No such file or directory while getting initial credentials"

2017-02-24 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57039/#review166742 --- ambari-server/src/main/resources/common-services/KAFKA/0.9.0/kerb

Re: Review Request 56974: Services should be able to specify that credential store is always enabled

2017-02-24 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56974/#review166745 --- Fix it, then Ship it! In general, patch looks good to me. amb

Re: Review Request 57039: Restart Kafka broker failed after enabling kerberos with "kinit(v5): No such file or directory while getting initial credentials"

2017-02-24 Thread Robert Levas
> On Feb. 24, 2017, 1:10 p.m., Attila Doroszlai wrote: > > ambari-server/src/main/resources/common-services/KAFKA/0.9.0/kerberos.json, > > line 10 > > > > > > In Zookeeper + Kafka cluster (no HDFS) this will result i

Re: Review Request 56997: Use storm user principal instead of nimbus user principal for ranger audit

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

Re: Review Request 56997: Use storm user principal instead of nimbus user principal for ranger audit

2017-02-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56997/#review166753 --- Ship it! Ship It! - Robert Levas On Feb. 24, 2017, 2:28 p.m.

Re: Review Request 57004: AMBARI-20158 - Host filter is missing for admin user after upgrading to ambari 2.4.2 or above

2017-02-24 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57004/#review166755 --- Ship it! - Jaimin Jetly On Feb. 24, 2017, 2:27 a.m., Richard

Re: Review Request 57039: Restart Kafka broker failed after enabling kerberos with "kinit(v5): No such file or directory while getting initial credentials"

2017-02-24 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57039/ --- (Updated Feb. 24, 2017, 9 p.m.) Review request for Ambari, Attila Magyar, Laszl

Re: Review Request 57039: Restart Kafka broker failed after enabling kerberos with "kinit(v5): No such file or directory while getting initial credentials"

2017-02-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57039/#review166757 --- Ship it! Ship It! - Robert Levas On Feb. 24, 2017, 4 p.m., B

Re: Review Request 57039: Restart Kafka broker failed after enabling kerberos with "kinit(v5): No such file or directory while getting initial credentials"

2017-02-24 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57039/#review166758 --- Ship it! Ship It! - Attila Doroszlai On Feb. 24, 2017, 10 p.

Review Request 57051: Move spinner for Next and Back button in Kerberos Wizard

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

Re: Review Request 57051: Move spinner for Next and Back button in Kerberos Wizard

2017-02-24 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57051/#review166767 --- Ship it! Ship It! - Yusaku Sako On Feb. 24, 2017, 10:27 p.m.

Review Request 57053: LogSearch Integration for HostComponent resource errors should be logged at the DEBUG level

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

Re: Review Request 57053: LogSearch Integration for HostComponent resource errors should be logged at the DEBUG level

2017-02-24 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57053/#review166773 --- Ship it! Ship It! - Oliver Szabo On Feb. 24, 2017, 10:50 p.m

Re: Review Request 57051: Move spinner for Next and Back button in Kerberos Wizard

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

Re: Review Request 57053: LogSearch Integration for HostComponent resource errors should be logged at the DEBUG level

2017-02-24 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57053/#review166771 --- Ship it! Ship It! - Miklos Gergely On Feb. 24, 2017, 10:50 p

Review Request 57056: Log Search Portal is not working with HTTPS with it's own created Key Store

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

Re: Review Request 57056: Log Search Portal is not working with HTTPS with it's own created Key Store

2017-02-24 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57056/ --- (Updated Feb. 25, 2017, 1:06 a.m.) Review request for Ambari, Oliver Szabo and

Re: Review Request 57053: LogSearch Integration for HostComponent resource errors should be logged at the DEBUG level

2017-02-24 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57053/ --- (Updated Feb. 25, 2017, 1:11 a.m.) Review request for Ambari, Miklos Gergely an

Re: Review Request 57056: Log Search Portal is not working with HTTPS with it's own created Key Store

2017-02-24 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57056/#review166796 --- Ship it! Ship It! - Robert Nettleton On Feb. 25, 2017, 1:06

Re: Review Request 56974: Services should be able to specify that credential store is always enabled

2017-02-24 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56974/ --- (Updated Feb. 25, 2017, 1:29 a.m.) Review request for Ambari, Alejandro Fernand

Re: Review Request 56974: Services should be able to specify that credential store is always enabled

2017-02-24 Thread Sumit Mohanty
> On Feb. 24, 2017, 7:04 p.m., Sid Wagle wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/ServiceInfo.java, > > line 481 > > > > > > Minor nit: Null check not needed since it is primitive type,

Re: Review Request 56997: Use storm user principal instead of nimbus user principal for ranger audit

2017-02-24 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56997/ --- (Updated Feb. 25, 2017, 2:59 a.m.) Review request for Ambari, Miklos Gergely, M

Review Request 57060: HostCleaup.py throws exception and exits when hostcheck_custom_actions.result not present

2017-02-24 Thread Amruta Borkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57060/ --- Review request for Ambari, Di Li and Jonathan Hurley. Bugs: AMBARI-20192 ht

Review Request 57062: AMBARI-20194 - Enable/Disable UI objects related to managing alert targets based on user's privileges

2017-02-24 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57062/ --- Review request for Ambari, Jaimin Jetly and Yusaku Sako. Bugs: AMBARI-20194

  1   2   >