Review Request 58493: AMBARI-20768: Local Ambari user with no cluster role must not be able to access Logsearch UI

2017-04-17 Thread Keta Patel
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58493/ --- Review request for Ambari. Bugs: AMBARI-20768

Re: Review Request 58489: VersionAdvertised should be set to False by default in stack_tool.py

2017-04-17 Thread Madhuvanthi Radhakrishnan
> On April 18, 2017, 12:36 a.m., Alejandro Fernandez wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/stack_tools.py > > Line 52 (original), 52 (patched) > > > > > > Why would

Review Request 58490: AMBARI-20777 : AMS changes to use instanceId for cluster based segregation of data

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

Re: Review Request 58489: VersionAdvertised should be set to False by default in stack_tool.py

2017-04-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58489/#review172160 ---

Review Request 58489: VersionAdvertised should be set to False by default in stack_tool.py

2017-04-17 Thread Madhuvanthi Radhakrishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58489/ --- Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Jayush Luniya,

Re: Review Request 58483: Service Upgrade VDF Creates Host Version Entries For All Hosts With INSTALLING

2017-04-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58483/#review172158 --- Ship it! Ship It! - Alejandro Fernandez On April 17, 2017,

Re: Review Request 58483: Service Upgrade VDF Creates Host Version Entries For All Hosts With INSTALLING

2017-04-17 Thread Jonathan Hurley
> On April 17, 2017, 5:57 p.m., Alejandro Fernandez wrote: > > Thanks for the review. I'll fix the log statement and the comment. - Jonathan --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 58483: Service Upgrade VDF Creates Host Version Entries For All Hosts With INSTALLING

2017-04-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58483/#review172140 ---

Re: Review Request 58445: AMBARI-20738 - Cannot add repo for additional OS to an existing HDP version

2017-04-17 Thread Xi Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58445/#review172139 --- Ship it! Ship It! - Xi Wang On April 13, 2017, 6:56 p.m.,

Review Request 58483: Service Upgrade VDF Creates Host Version Entries For All Hosts With INSTALLING

2017-04-17 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58483/ --- Review request for Ambari, Dmytro Grinenko and Nate Cole. Bugs: AMBARI-20774

Re: Review Request 58426: AMBARI-19149. Code cleanup: unresolved references in javadoc

2017-04-17 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58426/#review172135 --- Ship it! Ship It! - Jonathan Hurley On April 13, 2017, 1:14

Re: Review Request 58407: HDP 3.0 TP - temporarily install RPMs hive2 and remove Pig dependency from Hive

2017-04-17 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58407/#review172119 --- Ship it! Ship It! - Sid Wagle On April 13, 2017, 12:45

Re: Review Request 56677: Kafka still showing deprecated property log.cleanup.interval.mins in Config Section of Kafka on Ambari

2017-04-17 Thread Bharat Viswanadham
> On April 17, 2017, 5:49 p.m., Alejandro Fernandez wrote: > > Pushed to trunk, > > commit 6806d38b8526f5229a6ed4a56a4492f2b2ab27c2 Thank you Alejandro Fernandez - Bharat --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 56677: Kafka still showing deprecated property log.cleanup.interval.mins in Config Section of Kafka on Ambari

2017-04-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56677/#review172107 --- Pushed to trunk, commit 6806d38b8526f5229a6ed4a56a4492f2b2ab27c2

Re: Review Request 58480: Files View: In non-HA environment, Files view executes with error when "List of NameNodes" property is cleared

2017-04-17 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58480/#review172101 --- Ship it! Ship It! - Di Li On April 17, 2017, 4:51 p.m.,

Re: Review Request 57451: The Installation of hbase-client failed

2017-04-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57451/#review172097 ---

Review Request 58480: Files View: In non-HA environment, Files view executes with error when "List of NameNodes" property is cleared

2017-04-17 Thread Sangeeta Ravindran
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58480/ --- Review request for Ambari, Di Li and DIPAYAN BHOWMICK. Bugs: AMBARI-20354

Re: Review Request 58438: AMBARI-20763. Update YARN's ATS configs 'apptimelineserver_heapsize' and 'yarn.timeline-service.entity-group-fs-store.app-cache-size' logic in 2.6.

2017-04-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58438/#review172096 ---

Re: Review Request 58437: AMBARI-20763. Update YARN's ATS configs 'apptimelineserver_heapsize' and 'yarn.timeline-service.entity-group-fs-store.app-cache-size' logic in 2.6.

2017-04-17 Thread Alejandro Fernandez
> On April 17, 2017, 4:39 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/HDP/2.6/services/YARN/configuration/yarn-env.xml > > Lines 28 (patched) > > > > > > Please also include these

Re: Review Request 58437: AMBARI-20763. Update YARN's ATS configs 'apptimelineserver_heapsize' and 'yarn.timeline-service.entity-group-fs-store.app-cache-size' logic in 2.6.

2017-04-17 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58437/#review172094 ---

Re: Review Request 58409: get_value_from_jmx constantly prints exception message in retry mechanism, which brings bad user experience

2017-04-17 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58409/#review172089 --- please close this RB request as the change is pushed to trunk as

Re: Review Request 58431: Update zookeeper.connect description

2017-04-17 Thread Di Li
> On April 13, 2017, 8:26 p.m., Di Li wrote: > > Ship It! > > Bharat Viswanadham wrote: > Hi Di, > Could you please help in commit these changes. please close this RB request as the change is pushed to trunk as

Re: Review Request 58409: get_value_from_jmx constantly prints exception message in retry mechanism, which brings bad user experience

2017-04-17 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58409/#review172084 --- Ship it! Ship It! - Di Li On April 13, 2017, 4:16 a.m.,

Re: Review Request 58411: AMBARI-20726: Aggregate alert does not show status properly at Maintenance Mode change

2017-04-17 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58411/#review172081 ---

Re: Review Request 58426: AMBARI-19149. Code cleanup: unresolved references in javadoc

2017-04-17 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58426/#review172080 --- Ship it! Ship It! - Nate Cole On April 13, 2017, 1:14 p.m.,

Re: Review Request 58426: AMBARI-19149. Code cleanup: unresolved references in javadoc

2017-04-17 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58426/#review172079 --- Ship it! Ship It! - Sandor Magyari On April 13, 2017, 5:14