Re: Review Request 48557: Fixed implementation of on-ambari-upgrade support. Patch 2: add logic for ambari-upgrade

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48557/#review137472 --- Ship it! Ship It! - Andrew Onischuk On June 14, 2016, 9:07

Re: Review Request 48460: Ambari Views logs should include information of view instance

> On June 9, 2016, 12:09 p.m., DIPAYAN BHOWMICK wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java, > > line 773 > > > > > > Why is this changed. Seems like a rebase issue.

Re: Review Request 48655: ATLAS conf dir needs to be present in all ATLAS hook deployed hosts

> On June 13, 2016, 8:29 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml, > > line 238 > > > > > > Does this need to change for

Re: Review Request 48557: Fixed implementation of on-ambari-upgrade support. Patch 2: add logic for ambari-upgrade

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48557/ --- (Updated June 14, 2016, 12:07 p.m.) Review request for Ambari, Alejandro

Re: Review Request 48545: View config- Allows remote cluster creation using a user having no permissions on the cluster

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48545/#review137476 --- Ship it! Ship It! - DIPAYAN BHOWMICK On June 10, 2016,

Re: Review Request 48266: Add explicit ambari-server log line indicating cluster creation complete

> On jún. 13, 2016, 4:51 du, Robert Nettleton wrote: > > ambari-server/src/main/java/org/apache/ambari/server/agent/HeartbeatProcessor.java, > > line 191 > > > > > > I'm a little concerned that this approach could

Re: Review Request 48266: Add explicit ambari-server log line indicating cluster creation complete

> On jún. 13, 2016, 9:23 du, Sumit Mohanty wrote: > > Does this handle UI based cluster deployments? > > Probably there is no good way for UI based deplyments to know if a specific > > request is a cluster create request or not. > > > > What happens after cluster creation is over - does the

Re: Review Request 48266: Add explicit ambari-server log line indicating cluster creation complete

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48266/#review137477 --- Hi Bob, Summit! Using events instead of polling continusly seems

Re: Review Request 48490: Last button in the log search pagination panel does not take the user to the last page and few more fixes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48490/#review137465 ---

Re: Review Request 48480: View Config- views created for 2nd remote cluster points to 1st remote cluster

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48480/#review137475 --- Ship it! Ship It! - DIPAYAN BHOWMICK On June 9, 2016, 8:43

Review Request 48681: Ambari views logs should be more verbose and should include info/error/debug logs at appropriate places

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48681/ --- Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Pallav Kulshreshtha,

Re: Review Request 48589: Fix HA enabled logic in the alerts

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48589/#review137494 --- I don't think we need to special case this. Consider the alert

Review Request 48683: One of RM failed to start after ambaru upgrade in secure cluster [Upgrade]

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48683/ --- Review request for Ambari and Dmitro Lisnichenko. Bugs: AMBARI-17216

Re: Review Request 48682: AMBARI-17215: Zeppelin Hive configuration change to JDBC

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48682/ --- (Updated June 14, 2016, 10:49 a.m.) Review request for Ambari, Alejandro

Re: Review Request 48683: One of RM failed to start after ambaru upgrade in secure cluster [Upgrade]

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48683/ --- (Updated June 14, 2016, 10:49 a.m.) Review request for Ambari and Dmitro

Re: Review Request 48683: One of RM failed to start after ambaru upgrade in secure cluster [Upgrade]

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48683/#review137479 --- Ship it! Ship It! - Dmitro Lisnichenko On June 14, 2016,

Re: Review Request 48516: SPNEGO keytab and principal configuration for HBase web UIs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48516/#review137497 --- What is the status of this review? - Nate Cole On June 9,

Re: Review Request 48683: One of RM failed to start after ambaru upgrade in secure cluster [Upgrade]

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48683/ --- (Updated June 14, 2016, 10:59 a.m.) Review request for Ambari and Dmitro

Re: Review Request 48683: One of RM failed to start after ambaru upgrade in secure cluster [Upgrade]

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48683/#review137482 --- Ship it! Ship It! - Dmitro Lisnichenko On June 14, 2016,

Review Request 48685: AMBARI-17218 Show message of Audit to DB Removal during upgrade for Ranger

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48685/ --- Review request for Ambari, Gautam Borad, Jonathan Hurley, Srimanth Gunturi, and

Re: Review Request 48629: AMBARI-17189 : Change in Atlas authorization from class based to value based

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48629/#review137500 --- Ship it! Ship It! - Jonathan Hurley On June 13, 2016, 6:47

Review Request 48682: AMBARI-17215: Zeppelin Hive configuration change to JDBC

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48682/ --- Review request for Ambari, Alejandro Fernandez, DIPAYAN BHOWMICK, Jayush

Review Request 48686: Tez View: Improve proxy logging

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48686/ --- Review request for Ambari, DIPAYAN BHOWMICK and Pallav Kulshreshtha. Bugs:

Re: Review Request 48654: Upgrading Ambari Causes Alert Fields Not To Save Thresholds

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48654/ --- (Updated Червень 14, 2016, 10:41 до полудня) Review request for Ambari, Dmytro

Re: Review Request 48654: Upgrading Ambari Causes Alert Fields Not To Save Thresholds

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48654/#review137478 --- Ship it! Ship It! - Dmytro Sen On Июнь 14, 2016, 10:41

Re: Review Request 48490: Last button in the log search pagination panel does not take the user to the last page and few more fixes

> On June 14, 2016, 8:18 a.m., Miklos Gergely wrote: > > ambari-logsearch/ambari-logsearch-portal/src/main/java/org/apache/ambari/logsearch/view/VUserConfig.java, > > line 35 > > > > > > Why not using camel case for

Re: Review Request 47954: AMBARI-16907 Ambari web UI does not auto-set all required properties when user toggles on the Yarn CPU isolation feature on the web UI

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47954/#review137506 --- Ship it! Ship It! - Andrew Onischuk On June 13, 2016, 2:39

Re: Review Request 48686: Tez View: Improve proxy logging

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48686/#review137513 --- Does logging need to change in any of other classes of the tez

Re: Review Request 48414: AMBARI-17118 Incorrect formated external url in ranger configuration - causes Namenode startup failure

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48414/#review137515 --- Ship it! Ship It! - Gautam Borad On June 13, 2016, 2:15

Re: Review Request 48516: SPNEGO keytab and principal configuration for HBase web UIs

> On June 14, 2016, 1:03 p.m., Nate Cole wrote: > > What is the status of this review? Hi Nate. I had made these changes preemptively -- I'm actually still waiting on the corresponding HBase changes to get merged in. I think it best if we wait for that one to get in before we make this change

Review Request 48689: Ability to add "javax.jdo.option.ConnectionPassword" to hive clients from ambari

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48689/ --- Review request for Ambari and Robert Levas. Bugs: AMBARI-17223

Re: Review Request 48589: Fix HA enabled logic in the alerts

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48589/ --- (Updated June 14, 2016, 2:15 p.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 48266: Add explicit ambari-server log line indicating cluster creation complete

> On jún. 14, 2016, 10:28 de, Sandor Magyari wrote: > > Hi Bob, Summit! > > > > Using events instead of polling continusly seems to be a better solution, > > may be would be better to listen to already existing events like > > ActionFinalReportReceivedEvent which is fired when a command

Review Request 48691: Removing and re-adding hosts makes database inconsitent

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48691/ --- Review request for Ambari, Laszlo Puskas, Robert Nettleton, Sandor Magyari,

Re: Review Request 48689: Ability to add "javax.jdo.option.ConnectionPassword" to hive clients from ambari

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48689/#review137514 --- Ship it! Ship It! - Robert Levas On June 14, 2016, 10:01

Re: Review Request 48640: Add SERVICE.VIEW_OPERATIONAL_LOGS authorization to SERVICE.ADMINISTRATOR role and above

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48640/#review137503 --- Ship it! Ship It! - Eugene Chekanskiy On June 13, 2016,

Review Request 48690: org.apache.ambari.server.state.cluster.ClusterTest.testTransitionNonReportableHost fails intermittently on trunk.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48690/ --- Review request for Ambari and Myroslav Papirkovskyy. Bugs: AMBARI-17227

Re: Review Request 48690: org.apache.ambari.server.state.cluster.ClusterTest.testTransitionNonReportableHost fails intermittently on trunk.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48690/#review137511 --- Ship it! Ship It! - Jonathan Hurley On June 14, 2016, 10:33

Re: Review Request 48691: Removing and re-adding hosts makes database inconsitent

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48691/#review137527 --- Ship it! Ship It! - Laszlo Puskas On June 14, 2016, 2:51

Re: Review Request 48691: Removing and re-adding hosts makes database inconsitent

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48691/#review137519 --- Ship it! Ship It! - Sandor Magyari On June 14, 2016, 2:51

Re: Review Request 48691: Removing and re-adding hosts makes database inconsitent

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48691/#review137520 --- Ship it! Ship It! - Robert Nettleton On June 14, 2016, 2:51

Re: Review Request 48691: Removing and re-adding hosts makes database inconsitent

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48691/#review137521 --- Ship it! Ship It! - Sebastian Toader On June 14, 2016, 4:51

Review Request 48702: Add ability to set GET request directives

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48702/ --- Review request for Ambari, Ajit Kumar, Di Li, Jonathan Hurley, and Nate Cole.

Re: Review Request 48698: AMBARI-17230 VDF: when registering version using Default Vers Def, need to set Name

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48698/#review137546 --- Ship it! Ship It! - Jaimin Jetly On June 14, 2016, 5:01

Re: Review Request 48460: Ambari Views logs should include information of view instance

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48460/ --- (Updated June 14, 2016, 5:08 p.m.) Review request for Ambari, DIPAYAN

Re: Review Request 47656: AMBARI-12885 - Dynamic stack extensions - install and upgrade support for custom services

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47656/ --- (Updated June 14, 2016, 5:47 p.m.) Review request for Ambari, Alexander

Review Request 48698: AMBARI-17230 VDF: when registering version using Default Vers Def, need to set Name

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48698/ --- Review request for Ambari, Jaimin Jetly, Nate Cole, Oleg Nechiporenko, Richard

Re: Review Request 48702: Add ability to set GET request directives

> On June 14, 2016, 3:24 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/api/services/RequestFactory.java, > > line 69 > > > > > > createReadRequest() ? This is consistent with

Re: Review Request 47656: AMBARI-12885 - Dynamic stack extensions - install and upgrade support for custom services

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47656/ --- (Updated June 14, 2016, 7:05 p.m.) Review request for Ambari, Alexander

Review Request 48706: 'ambari-server --version' command does not show build number

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48706/ --- Review request for Ambari, Dmitro Lisnichenko and Dmytro Sen. Repository:

Review Request 48707: Atlas Server Web UI alert

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48707/ --- Review request for Ambari and John Speidel. Bugs: AMBARI-17005

Re: Review Request 48702: Add ability to set GET request directives

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48702/ --- (Updated June 14, 2016, 2:57 p.m.) Review request for Ambari, Ajit Kumar, Di

Re: Review Request 48702: Add ability to set GET request directives

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48702/#review137568 --- Can you give an example of a read directive?

Re: Review Request 47656: AMBARI-12885 - Dynamic stack extensions - install and upgrade support for custom services

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47656/ --- (Updated June 14, 2016, 9:09 p.m.) Review request for Ambari, Alexander

Re: Review Request 47656: AMBARI-12885 - Dynamic stack extensions - install and upgrade support for custom services

> On June 3, 2016, 9:46 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql, line 32 > > > > > > Let's be consistent with the name, UQ_* Just noting that the code was

Re: Review Request 48685: AMBARI-17218 Show message of Audit to DB Removal during upgrade for Ranger

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48685/ --- (Updated June 15, 2016, 5:13 a.m.) Review request for Ambari, Gautam Borad,

Re: Review Request 48685: AMBARI-17218 Show message of Audit to DB Removal during upgrade for Ranger

> On June 14, 2016, 1:03 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/RangerAuditDbCheck.java, > > line 35 > > > > > > Maybe make a new UpgradeCheckGroup for this -

Re: Review Request 48670: Return well formatted error response while deleting host with clients installed.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48670/#review137660 ---

Re: Review Request 48706: 'ambari-server --version' command does not show build number

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48706/#review137654 --- Ship it! Ship It! - Sumit Mohanty On June 14, 2016, 7:41

Re: Review Request 48395: AMBARI-17027: Metrics Collector API: Introduce basic series aggregation functions

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48395/#review137656 --- Ship it! Backend changes look ok. I would recommend a 'Ship

Re: Review Request 48702: Add ability to set GET request directives

> On June 14, 2016, 3:24 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/api/services/RequestFactory.java, > > line 69 > > > > > > createReadRequest() ? > > Robert Levas wrote: >

Re: Review Request 48702: Add ability to set GET request directives

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48702/ --- (Updated June 14, 2016, 7:12 p.m.) Review request for Ambari, Ajit Kumar, Di

Re: Review Request 48710: atlas.authentication.principal should be set to atlas/_h...@example.com

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48710/#review137620 --- Ship it! Ship It! - Alejandro Fernandez On June 14, 2016,

Re: Review Request 48697: AMBARI-17229. Add config 'tez.lib.uris' in tez_hive2/tez-site's pointing to tez_hive2 path jar.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48697/#review137619 --- Ship it! Ship It! - Alejandro Fernandez On June 14, 2016,

Re: Review Request 48702: Add ability to set GET request directives

> On June 14, 2016, 3:24 p.m., Nate Cole wrote: > > Can you give an example of a read directive? This is for a future enhancement I need to make. With the addition of the `when` clause for a Kerberos descriptor identity, we are able to optionally include Kerberos identities depending on what

Review Request 48708: Namenode start step failed during EU with RetriableException

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48708/ --- Review request for Ambari, Alejandro Fernandez, Dmitro Lisnichenko, and Nate

Re: Review Request 48708: Namenode start step failed during EU with RetriableException

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48708/#review137586 ---

Review Request 48709: Nothing shows on the role section when user has the same role as its group

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48709/ --- Review request for Ambari and Zhe (Joe) Wang. Bugs: AMBARI-17239

Re: Review Request 48709: Nothing shows on the role section when user has the same role as its group

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48709/#review137598 --- Ship it! Ship It! - Zhe (Joe) Wang On June 14, 2016, 10:39

Re: Review Request 48702: Add ability to set GET request directives

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48702/#review137621 --- Ship it! Ship It! - Alejandro Fernandez On June 14, 2016,

Re: Review Request 48683: One of RM failed to start after ambaru upgrade in secure cluster [Upgrade]

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48683/#review137631 ---

Re: Review Request 48670: Return well formatted error response while deleting host with clients installed.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48670/#review137629 ---

Re: Review Request 48707: Atlas Server Web UI alert

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48707/#review137591 --- Ship it! Ship It! - John Speidel On June 14, 2016, 7:58

Re: Review Request 48710: atlas.authentication.principal should be set to atlas/_h...@example.com

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48710/#review137613 --- Ship it! Ship It! - Robert Levas On June 14, 2016, 7:03

Review Request 48710: atlas.authentication.principal should be set to atlas/_h...@example.com

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48710/ --- Review request for Ambari and Robert Levas. Bugs: AMBARI-17242

Re: Review Request 48708: Namenode start step failed during EU with RetriableException

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48708/#review137623 --- Ship it! Ship It! - Alejandro Fernandez On June 14, 2016,

Re: Review Request 48708: Namenode start step failed during EU with RetriableException

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48708/#review137599 --- Ship it! Ship It! - Nate Cole On June 14, 2016, 5:33 p.m.,

Re: Review Request 48697: AMBARI-17229. Add config 'tez.lib.uris' in tez_hive2/tez-site's pointing to tez_hive2 path jar.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48697/#review137615 --- Ship it! Ship It! - Sumit Mohanty On June 14, 2016, 11:12

Re: Review Request 47656: AMBARI-12885 - Dynamic stack extensions - install and upgrade support for custom services

> On June 3, 2016, 9:46 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql, line 32 > > > > > > Let's be consistent with the name, UQ_* > > Tim Thorpe wrote: > Just

Re: Review Request 48636: Zeppelin Views are not working with Custom and Remote cluster view configuration

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48636/#review137626 --- Ship it! Ship It! - Alejandro Fernandez On June 13, 2016,

Re: Review Request 48670: Return well formatted error response while deleting host with clients installed.

> On June 14, 2016, 11:54 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/HostResourceProvider.java, > > line 930 > > > > > > How is this a fix? > >

Re: Review Request 48589: Fix HA enabled logic in the alerts

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48589/#review137632 --- Ship it! Ship It! - Jonathan Hurley On June 14, 2016, 10:15

Re: Review Request 48335: Zeppelin service: Update default zeppelin_pid_dir to /var/run/zeppelin

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48335/#review137627 --- Ship it! Ship It! - Alejandro Fernandez On June 7, 2016,

Review Request 48697: AMBARI-17229. Add config 'tez.lib.uris' in tez_hive2/tez-site's pointing to tez_hive2 path jar.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48697/ --- Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. Bugs:

Re: Review Request 48697: AMBARI-17229. Add config 'tez.lib.uris' in tez_hive2/tez-site's pointing to tez_hive2 path jar.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48697/ --- (Updated June 14, 2016, 11:12 p.m.) Review request for Ambari, Alejandro

Re: Review Request 48516: SPNEGO keytab and principal configuration for HBase web UIs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48516/#review137637 --- Ship it! Ship It! - Alejandro Fernandez On June 9, 2016, 11