Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

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

Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

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

Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

> On July 26, 2016, 3:35 p.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/RANGER/0.6.0/kerberos.json, > > line 17 > > > > > > Would Solr not be Kerberized if Kerberos is enabled for the

Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50445/#review143667 --- Is Kerberized External Solr + Unkerberized Cluster a valid

Re: Review Request 50479: AMBARI-17909 AMS Storm Sink: apply change of Storm metrics improvement - worker level aggregation

> On 7 27, 2016, 2:10 오전, Sriharsha Chintalapani wrote: > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/configuration/storm-site.xml, > > line 54 > > > > > > we should make this as true

Re: Review Request 50479: AMBARI-17909 AMS Storm Sink: apply change of Storm metrics improvement - worker level aggregation

> On 7 27, 2016, 2:31 오전, Jungtaek Lim wrote: > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/configuration/storm-site.xml, > > line 54 > > > > > > To tell the truth I don't understand how

Re: Review Request 50479: AMBARI-17909 AMS Storm Sink: apply change of Storm metrics improvement - worker level aggregation

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50479/ --- (Updated 7 27, 2016, 2:52 오전) Review request for Ambari, Aravindan Vijayan,

Re: Review Request 50479: AMBARI-17909 AMS Storm Sink: apply change of Storm metrics improvement - worker level aggregation

> On July 27, 2016, 2:31 a.m., Jungtaek Lim wrote: > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/configuration/storm-site.xml, > > line 54 > > > > > > To tell the truth I don't understand

Re: Review Request 50479: AMBARI-17909 AMS Storm Sink: apply change of Storm metrics improvement - worker level aggregation

> On 7 27, 2016, 2:10 오전, Sriharsha Chintalapani wrote: > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/configuration/storm-site.xml, > > line 54 > > > > > > we should make this as true

Review Request 50479: AMBARI-17909 AMS Storm Sink: apply change of Storm metrics improvement - worker level aggregation

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50479/ --- Review request for Ambari, Aravindan Vijayan, Sriharsha Chintalapani, and Sid

Re: Review Request 50431: AMBARI-17896: Fix falcon hook classname and classpath for atlas

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50431/ --- (Updated July 26, 2016, 6:07 p.m.) Review request for Ambari and Alejandro

Re: Review Request 50476: UI does not allow to delete service is some host components are in INSTALLED_FAILED state

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50476/#review143648 --- Ship it! Ship It! - Sumit Mohanty On July 27, 2016, 12:32

Review Request 50476: UI does not allow to delete service is some host components are in INSTALLED_FAILED state

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50476/ --- Review request for Ambari, Zhe (Joe) Wang, Sumit Mohanty, and Yusaku Sako.

Review Request 50475: AMBARI-17778 Add usage information in ambari-server script

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50475/ --- Review request for Ambari, Nate Cole, Sebastian Toader, Sid Wagle, and Yusaku

Re: Review Request 50461: Downgrade Schedules Manual & Configuration Tasks For Components Which Didn't Upgrade

> On July 26, 2016, 4:36 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/TaskWrapperBuilder.java, > > lines 59-61 > > > > > > Won't all CONFIGURE tasks have

Re: Review Request 50461: Downgrade Schedules Manual & Configuration Tasks For Components Which Didn't Upgrade

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50461/#review143629 --- Ship it! Ship It! - Jonathan Hurley On July 26, 2016, 4:17

Review Request 50470: AMBARI-17907 On AMS collector move the webapp address does not change

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50470/ --- Review request for Ambari, Jaimin Jetly, Richard Zang, Vivek Ratnavel

Re: Review Request 50431: AMBARI-17896: Fix falcon hook classname and classpath for atlas

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

Re: Review Request 50431: AMBARI-17896: Fix falcon hook classname and classpath for atlas

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

Re: Review Request 50431: AMBARI-17896: Fix falcon hook classname and classpath for atlas

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50431/#review143616 --- Ship it! Ship It! - Alejandro Fernandez On July 26, 2016,

Re: Review Request 50461: Downgrade Schedules Manual & Configuration Tasks For Components Which Didn't Upgrade

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50461/#review143615 --- Ship it! Ship It! - Alejandro Fernandez On July 26, 2016,

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

> On July 26, 2016, 8:41 p.m., Robert Levas wrote: > > Committed to trunk > > ``` > > commit 124f48ef899fddb6bdb96ebea9aa3a6a1a6adbca > > Author: Anita Jebaraj > > Date: Tue Jul 26 16:25:30 2016 -0400 > > ``` > > > > Committed to branch-2.4 > > ``` > > commit

Re: Review Request 50461: Downgrade Schedules Manual & Configuration Tasks For Components Which Didn't Upgrade

> On July 26, 2016, 4:36 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/TaskWrapperBuilder.java, > > lines 59-61 > > > > > > Won't all CONFIGURE tasks have

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50047/#review143608 --- Committed to trunk ``` commit

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50047/#review143607 --- Ship it! Ship It! - Jonathan Hurley On July 26, 2016, 2:15

Re: Review Request 50461: Downgrade Schedules Manual & Configuration Tasks For Components Which Didn't Upgrade

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

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

> On July 26, 2016, 8:09 p.m., Nate Cole wrote: > > Ship It! Thank you Nate - Anita --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50047/#review143599

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

> On July 25, 2016, 10:21 p.m., Robert Levas wrote: > > Ship It! > > Robert Levas wrote: > Thanks for making those changes. > > Anita Jebaraj wrote: > can you please help in pushing the code > > Robert Levas wrote: > sure... working on it now. > > Robert Levas wrote: >

Review Request 50461: Downgrade Schedules Manual & Configuration Tasks For Components Which Didn't Upgrade

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

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

> On July 26, 2016, 11:18 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelper.java, > > line 251 > > > > > > May want to try/catch around these

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50047/#review143599 --- Ship it! Ship It! - Nate Cole On July 26, 2016, 2:15 p.m.,

Re: Review Request 50431: AMBARI-17896: Fix falcon hook classname and classpath for atlas

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50431/ --- (Updated July 26, 2016, 1:05 p.m.) Review request for Ambari and Alejandro

Re: Review Request 50431: AMBARI-17896: Fix falcon hook classname and classpath for atlas

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

Re: Review Request 50395: Create configuration flag to prevent changing of directory permissions

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50395/ --- (Updated July 26, 2016, 7:49 p.m.) Review request for Ambari, Andrew Onischuk

Re: Review Request 50350: Config changes for Atlas in HDP 2.5 related to atlas.rest.address, atlas.cluster.name, etc

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50350/#review143584 --- Pushed to trunk, commit e424b5b538537ea16f7d8cc24899c7a4ff5864a0

Re: Review Request 50350: Config changes for Atlas in HDP 2.5 related to atlas.rest.address, atlas.cluster.name, etc

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50350/ --- (Updated July 26, 2016, 6:48 p.m.) Review request for Ambari, Di Li, Jonathan

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

> On July 26, 2016, 3:18 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelper.java, > > line 251 > > > > > > May want to try/catch around these

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

> On July 26, 2016, 3:18 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelper.java, > > line 240 > > > > > > Should be ReplaceValue (capitalize

Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

> On July 26, 2016, 3:35 p.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/RANGER/0.6.0/kerberos.json, > > line 17 > > > > > > Would Solr not be Kerberized if Kerberos is enabled for the

Re: Review Request 50448: HBase coprocessor classes related properties in hbase-site are empty after Ambari upgrade from 2.1.2 to 2.4.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50448/#review143564 --- Ship it! Ship It! - Vitalyi Brodetskyi On Липень 26, 2016,

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

> On July 26, 2016, 11:18 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelper.java, > > line 251 > > > > > > May want to try/catch around these

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

> On July 26, 2016, 11:18 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelper.java, > > line 251 > > > > > > May want to try/catch around these

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

> On July 26, 2016, 11:18 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelper.java, > > line 248 > > > > > > Use StringBuilder, which doesn't

Re: Review Request 50350: Config changes for Atlas in HDP 2.5 related to atlas.rest.address, atlas.cluster.name, etc

> On July 26, 2016, 2:23 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hcat.py, > > line 80 > > > > > > It would be nice to extract these to a

Re: Review Request 50350: Config changes for Atlas in HDP 2.5 related to atlas.rest.address, atlas.cluster.name, etc

> On July 26, 2016, 1:34 a.m., Sumit Mohanty wrote: > > ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py, > > line 217 > > > > > > Any reason its in 2.3 as it seems to be only used in 2.5.

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

> On July 26, 2016, 3:18 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelper.java, > > line 248 > > > > > > Use StringBuilder, which doesn't

Review Request 50448: HBase coprocessor classes related properties in hbase-site are empty after Ambari upgrade from 2.1.2 to 2.4.0

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

Re: Review Request 50409: Adding Services After A Stack/Ambari Upgrade Shows Empty Required Values

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50409/#review143548 --- Ship it! Ship It! - Dmitro Lisnichenko On July 26, 2016,

Re: Review Request 50409: Adding Services After A Stack/Ambari Upgrade Shows Empty Required Values

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50409/ --- (Updated July 26, 2016, 11:52 a.m.) Review request for Ambari, Dmytro

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

> On July 26, 2016, 11:18 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelper.java, > > line 248 > > > > > > Use StringBuilder, which doesn't

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

> On July 26, 2016, 11:18 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/VariableReplacementHelper.java, > > line 248 > > > > > > Use StringBuilder, which doesn't

Re: Review Request 50350: Config changes for Atlas in HDP 2.5 related to atlas.rest.address, atlas.cluster.name, etc

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50350/#review143541 --- Ship it! Ship It! - Tim Thorpe On July 26, 2016, 1:22 a.m.,

Re: Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

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

Re: Review Request 50350: Config changes for Atlas in HDP 2.5 related to atlas.rest.address, atlas.cluster.name, etc

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50350/#review143539 --- Ship it! Ship It! - Di Li On July 26, 2016, 1:22 a.m.,

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

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

Re: Review Request 50350: Config changes for Atlas in HDP 2.5 related to atlas.rest.address, atlas.cluster.name, etc

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50350/#review143537 --- Ship it! Ship It! - Nate Cole On July 25, 2016, 9:22 p.m.,

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

> On July 25, 2016, 10:21 p.m., Robert Levas wrote: > > Ship It! > > Robert Levas wrote: > Thanks for making those changes. > > Anita Jebaraj wrote: > can you please help in pushing the code > > Robert Levas wrote: > sure... working on it now. > > Robert Levas wrote: >

Re: Review Request 50446: Check HBase is failed after Upgrade from 2.1.2 to 2.4.0.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50446/#review143529 --- Ship it! Ship It! - Dmitro Lisnichenko On July 26, 2016,

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

> On July 25, 2016, 6:21 p.m., Robert Levas wrote: > > Ship It! > > Robert Levas wrote: > Thanks for making those changes. > > Anita Jebaraj wrote: > can you please help in pushing the code > > Robert Levas wrote: > sure... working on it now. Actually, we only have 1 +1. Can you

Review Request 50445: AMBARI-17902 Config changes to support external solr and internal solr for Ranger

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50445/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jaimin Jetly,

Re: Review Request 50140: AMBARI-17771 : Update alert definition for Ranger Admin after kerberos changes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50140/ --- (Updated July 26, 2016, 2:46 p.m.) Review request for Ambari, Alejandro

Review Request 50446: Check HBase is failed after Upgrade from 2.1.2 to 2.4.0.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50446/ --- Review request for Ambari, Dmitro Lisnichenko, Sid Wagle, and Vitalyi

Re: Review Request 48734: App timeline Server start fails on enabling HA because namenode is in safemode

> On July 13, 2016, 7:02 a.m., Sebastian Toader wrote: > > I think this is rather generic problem that needs to be handled in > > *HdfsResourceJar* and *HdfsResourceWebHDFS (WebHDFSUtil)*. > > > > These are the classes that carry out the HDFS operations. All retryable > > operations (e.g.

Re: Review Request 48734: App timeline Server start fails on enabling HA because namenode is in safemode

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48734/#review143523 --- Ship it! Ship It! - Sandor Magyari On June 17, 2016, 10:45

Re: Review Request 50047: Kafka listeners property does not show SASL_PLAINTEXT protocol when Kerberos is enabled

> On July 25, 2016, 10:21 p.m., Robert Levas wrote: > > Ship It! > > Robert Levas wrote: > Thanks for making those changes. can you please help in pushing the code - Anita --- This is an automatically generated e-mail. To reply,

Re: Review Request 50396: Bad error message if user has not performed ambari-server setup for jdbc driver before configuring custom DB

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50396/#review143519 --- Ship it! Ship It! - Robert Levas On July 26, 2016, 8:40

Re: Review Request 50409: Adding Services After A Stack/Ambari Upgrade Shows Empty Required Values

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50409/#review143515 --- Ship it! Ship It! - Nate Cole On July 25, 2016, 5:04 p.m.,

Re: Review Request 50396: Bad error message if user has not performed ambari-server setup for jdbc driver before configuring custom DB

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50396/#review143514 --- Ship it! Ship It! - Dmytro Sen On Июль 26, 2016, 12:40

Re: Review Request 50396: Bad error message if user has not performed ambari-server setup for jdbc driver before configuring custom DB

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50396/ --- (Updated Липень 26, 2016, 12:40 після полудня) Review request for Ambari,

Re: Review Request 50407: Installing Components After Registering A New Repository Uses The Wrong Version

> On July 25, 2016, 10:31 p.m., Jonathan Hurley wrote: > > ambari-common/src/main/python/resource_management/libraries/script/script.py, > > lines 369-372 > > > > > > Is there any reason we shouldn't just

Re: Review Request 49387: Support password type for custom properties

> On July 25, 2016, 9:08 p.m., Alexandr Antonenko wrote: > > Ship It! > > Keta Patel wrote: > Thank you Alexandr! > Could you please help with pushing in the changes? done - Alexandr --- This is an automatically generated

Re: Review Request 50406: HBase OOM while booting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50406/#review143495 --- Ship it! Ship It! - Dmytro Sen On Июль 25, 2016, 7:53 п.п.,

Re: Review Request 50297: AMBARI-17788: Refactor spooler code in OutputHDFSFile to be reusable for OutputS3File

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50297/#review143489 --- Ship it! Ship It! - Dharmesh Makwana On July 26, 2016, 5:55

Re: Review Request 50297: AMBARI-17788: Refactor spooler code in OutputHDFSFile to be reusable for OutputS3File

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50297/#review143486 --- Ship it! Ship It! - Hayat Behlim On July 26, 2016, 5:55