Re: Review Request 47586: Log Feeder Global exclusion default filter configuration is not effective in the log search indexing.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47586/ --- (Updated June 2, 2016, 5:42 a.m.) Review request for Ambari, Alejandro Fernande

Review Request 48162: 16171 Addendum2 for stackadvisor with Phoenix Query Server kerberos configuration

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

Re: Review Request 48157: AMBARI-16994: Ambari Server Upgrade should always update stack_features and stack_tools config properties

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48157/ --- (Updated June 2, 2016, 1:25 a.m.) Review request for Ambari, Alejandro Fernande

Re: Review Request 48125: AMBARI-16984 Add changes for Ranger Service to get previous_jdbc_jar_name

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48125/#review135889 --- ambari-server/src/main/resources/common-services/RANGER/0.4.0/pac

Re: Review Request 47705: AMBARI-16731 Should close file object in security.py

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47705/#review135888 --- Could anyone review this? I think this isn't a difficult patch. -

Re: Review Request 48134: Remove logsearch-solr-client dependency from Atlas

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48134/ --- (Updated June 2, 2016, 12:13 a.m.) Review request for Ambari, Robert Nettleton,

Re: Review Request 47923: clean up import * for SPARK service scripts in common-services

> On June 1, 2016, 2:53 a.m., Jayush Luniya wrote: > > Patch doesnt apply cleanly. Can you rebase the patch? > > Juanjo Marron wrote: > Patch submitted to https://issues.apache.org/jira/browse/AMBARI-16797. > Waiting for Hadoop QA review Patch applied cleanly by Hadoop QA Hadoop QA ad

Re: Review Request 46727: [AMBARI-15612] Add Livy to HDP 2.5 as slave component of Spark

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

Re: Review Request 48157: AMBARI-16994: Ambari Server Upgrade should always update stack_features and stack_tools config properties

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48157/#review135883 --- ambari-server/src/test/python/stacks/utils/RMFTestCase.py (line 1

Re: Review Request 48157: AMBARI-16994: Ambari Server Upgrade should always update stack_features and stack_tools config properties

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48157/ --- (Updated June 1, 2016, 10:59 p.m.) Review request for Ambari, Alejandro Fernand

Re: Review Request 48157: AMBARI-16994: Ambari Server Upgrade should always update stack_features and stack_tools config properties

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48157/#review135882 --- Ship it! ambari-common/src/main/python/resource_management/lib

Re: Review Request 48134: Remove logsearch-solr-client dependency from Atlas

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48134/#review135881 --- Ship it! Ship It! - Sumit Mohanty On June 1, 2016, 4:58 p.m.

Review Request 48157: AMBARI-16994: Ambari Server Upgrade should always update stack_features and stack_tools config properties

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

Re: Review Request 48156: Misc fixes related to slider while enabling interactive query

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48156/#review135880 --- Ship it! Ship It! - Zhe (Joe) Wang On June 1, 2016, 9:36 p.m

Re: Review Request 48156: Misc fixes related to slider while enabling interactive query

> On June 1, 2016, 9:40 p.m., Zhe (Joe) Wang wrote: > > ambari-web/app/controllers/wizard/step7/assign_master_controller.js, line > > 161 > > > > > > Typo? > > Jaimin Jetly wrote: > This is intended change to a

Re: Review Request 48123: Update default ambari view read timeout to a larger value

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

Re: Review Request 48147: Fix metric sink + rename misnamed zk-connect-string variables.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48147/#review135877 --- Ship it! Ship It! - Oliver Szabo On June 1, 2016, 7 p.m., Mi

Re: Review Request 48156: Misc fixes related to slider while enabling interactive query

> On June 1, 2016, 9:41 p.m., Zhe (Joe) Wang wrote: > > ambari-web/test/controllers/wizard/step7/assign_master_controller_test.js, > > line 152 > > > > > > Typo? unit test change needed to accomodate the change mad

Re: Review Request 48156: Misc fixes related to slider while enabling interactive query

> On June 1, 2016, 9:40 p.m., Zhe (Joe) Wang wrote: > > ambari-web/app/controllers/wizard/step7/assign_master_controller.js, line > > 161 > > > > > > Typo? This is intended change to address scenario 1 stated in de

Review Request 48156: Misc fixes related to slider while enabling interactive query

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

Re: Review Request 48156: Misc fixes related to slider while enabling interactive query

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48156/#review135868 --- ambari-web/test/controllers/wizard/step7/assign_master_controller

Re: Review Request 48156: Misc fixes related to slider while enabling interactive query

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48156/#review135867 --- ambari-web/app/controllers/wizard/step7/assign_master_controller.

Re: Review Request 47923: clean up import * for SPARK service scripts in common-services

> On June 1, 2016, 2:53 a.m., Jayush Luniya wrote: > > Patch doesnt apply cleanly. Can you rebase the patch? Patch submitted to https://issues.apache.org/jira/browse/AMBARI-16797. Waiting for Hadoop QA review - Juanjo --- This is an aut

Re: Review Request 48155: Recommend hawq_rm_nvcore_limit_perseg in HAWQ service advisor

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48155/#review135863 --- Ship it! Ship It! - bhuvnesh chaudhary On June 1, 2016, 9:02

Re: Review Request 47653: clean up import * for OOZIE service scripts in common-services

> On June 1, 2016, 2:33 a.m., Jayush Luniya wrote: > > Patch doesnt apply cleanly. Can you rebase the patch and resolve conflicts? New patch submitted to https://issues.apache.org/jira/browse/AMBARI-16672. Waiting for HAdoop QA review - Juanjo

Re: Review Request 48154: Incorrect ports in WebHDFS HA config in Knox

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48154/#review135860 --- Fix it, then Ship it! ambari-server/src/main/resources/common-

Review Request 48155: Recommend hawq_rm_nvcore_limit_perseg in HAWQ service advisor

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48155/ --- Review request for Ambari, Alexander Denissov, bhuvnesh chaudhary, and Lav Jain.

Re: Review Request 47858: Cache service advisors when stack advisor is loaded

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47858/#review135854 --- Ship it! Ship It! - Jayush Luniya On May 27, 2016, 7:11 p.m.

Re: Review Request 47880: AMBARI-16890 Updating Ambari configs changes for latest Ranger configs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47880/#review135852 --- Ship it! Ship It! - Alejandro Fernandez On May 30, 2016, 9:2

Re: Review Request 48143: Missing import check_process_status for YARN nodemanager service

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48143/#review135851 --- Ship it! Ship It! - Jayush Luniya On June 1, 2016, 6:26 p.m.

Review Request 48154: Incorrect ports in WebHDFS HA config in Knox

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48154/ --- Review request for Ambari, Alejandro Fernandez and Dmytro Sen. Bugs: AMBARI-138

Re: Review Request 48134: Remove logsearch-solr-client dependency from Atlas

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48134/#review135849 --- Ship it! Looks good to me. Thanks for doing that! - Tom Beerb

Re: Review Request 47879: AMBARI-16807: After enabling HTTPS for HDFS, Data Node JVM Metrics on HDFS Heatmaps show NA

> On May 26, 2016, 3:11 p.m., Dmytro Sen wrote: > > Ship It! > > Qin Liu wrote: > Thank you so much Dmytro! Hi Sid, Could you please review the patch at your earliest convenience? Thank you in advance! - Qin --- This is an automat

Re: Review Request 48147: Fix metric sink + rename misnamed zk-connect-string variables.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48147/ --- (Updated June 1, 2016, 6:57 p.m.) Review request for Ambari, Oliver Szabo, Robe

Review Request 48147: Fix metric sink

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48147/ --- Review request for Ambari, Oliver Szabo, Robert Nettleton, and Sumit Mohanty. B

Re: Review Request 48036: Service name shown instead of Host name on popup

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48036/#review135840 --- Ship it! Ship It! - Nate Cole On June 1, 2016, 12:01 p.m., D

Re: Review Request 47963: clean up import * for YARN service

> On June 1, 2016, 5:49 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/nodemanager.py, > > line 91 > > > > > > @Juanjo, it looks like check_process_stat

Re: Review Request 47586: Log Feeder Global exclusion default filter configuration is not effective in the log search indexing.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47586/#review135836 --- Ship it! Ship It! - Robert Nettleton On June 1, 2016, 6:20 a

Re: Review Request 48143: Missing import check_process_status for YARN nodemanager service

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48143/ --- (Updated June 1, 2016, 6:26 p.m.) Review request for Ambari and Jayush Luniya.

Review Request 48143: Missing import check_process_status for HDFS nodemanager service

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48143/ --- Review request for Ambari and Jayush Luniya. Bugs: AMBARI-16849 https://iss

Re: Review Request 48139: Required empty config is present after Upgrade from 2.1.2.1 to 2.4.0.0 for HIVE and HBASE

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48139/#review135828 --- Ship it! Ship It! - Dmytro Sen On Июнь 1, 2016, 6:06 п.п., V

Review Request 48139: Required empty config is present after Upgrade from 2.1.2.1 to 2.4.0.0 for HIVE and HBASE

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48139/ --- Review request for Ambari, Dmytro Sen, Sandor Magyari, and Sumit Mohanty. Bugs:

Re: Review Request 48122: History tab takes long to populate when there is more entry in history table.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48122/#review135825 --- Ship it! Ship It! - Rohit Choudhary On June 1, 2016, 1:53 p.

Re: Review Request 47963: clean up import * for YARN service

> On June 1, 2016, 5:49 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/nodemanager.py, > > line 91 > > > > > > @Juanjo, it looks like check_process_stat

Re: Review Request 47963: clean up import * for YARN service

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47963/#review135820 --- ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/p

Re: Review Request 48105: AMBARI-16977 VDF: couple UI bugs in Install Wiz

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48105/#review135816 --- Ship it! Ship It! - Jaimin Jetly On June 1, 2016, 12:54 a.m.

Re: Review Request 48091: Update derivation of hawq_rm_memory_limit_perseg

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48091/#review135814 --- Ship it! Ship It! - Lav Jain On May 31, 2016, 8:31 p.m., bhu

Review Request 48136: AMBARI-16973 Print out configType has malformed json text as part of the ambari server startup error handling

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

Review Request 48134: Remove logsearch-solr-client dependency from Atlas

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48134/ --- Review request for Ambari, Robert Nettleton, Sumit Mohanty, and Tom Beerbower.

Re: Review Request 47964: clean up import * for ZOOKEEPER service

> On June 1, 2016, 3:07 a.m., Jayush Luniya wrote: > > Ship It! Thanks for the reviews and fo commitying to trunk - Juanjo --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47964/#review135757

Re: Review Request 48135: Ambari-server fails to start after reboot on Ubuntu

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48135/#review135810 --- Ship it! Ship It! - Dmitro Lisnichenko On June 1, 2016, 7:45

Re: Review Request 48135: Ambari-server fails to start after reboot on Ubuntu

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48135/ --- (Updated June 1, 2016, 4:45 p.m.) Review request for Ambari and Dmitro Lisniche

Review Request 48135: Ambari-server fails to start after reboot on Ubuntu

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

Re: Review Request 48121: YARN default configs are invalid

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48121/#review135809 --- Ship it! Ship It! - Vitalyi Brodetskyi On Червень 1, 2016, 1

Re: Review Request 48073: Exception while detecting JMX protocol for HISTORYSERVER

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48073/#review135808 --- Ship it! Ship It! - Vitalyi Brodetskyi On Червень 1, 2016, 4

Re: Review Request 48073: Exception while detecting JMX protocol for HISTORYSERVER

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48073/ --- (Updated June 1, 2016, 4:26 p.m.) Review request for Ambari and Vitalyi Brodets

Re: Review Request 48036: Service name shown instead of Host name on popup

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

Re: Review Request 47586: Log Feeder Global exclusion default filter configuration is not effective in the log search indexing.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47586/#review135797 --- Ship it! Ship It! - Miklos Gergely On June 1, 2016, 6:20 a.m

Re: Review Request 48063: Redundant info under config tab for falcon

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48063/#review135796 --- Ship it! Ship It! - Vitalyi Brodetskyi On Червень 1, 2016, 1

Re: Review Request 48122: History tab takes long to populate when there is more entry in history table.

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

Re: Review Request 48088: Stack VDF should ignore unsupported OS

> On June 1, 2016, 8:59 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/LatestRepoCallable.java, > > line 197 > > > > > > Would contains() be clearer? Thanks for re

Re: Review Request 47428: Changes to Phoenix QueryServer Kerberos configuration

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47428/#review135794 --- Ship it! Ship It! - Robert Levas On May 27, 2016, 6:48 p.m.,

Re: Review Request 48124: Host checkboxes getting unselected when user clicks on search host button and few minor issues

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48124/#review135793 --- Ship it! Ship It! - Oliver Szabo On June 1, 2016, 12:13 p.m.

Re: Review Request 47428: Changes to Phoenix QueryServer Kerberos configuration

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47428/#review135792 --- Ship it! Ship It! - Nate Cole On May 27, 2016, 6:48 p.m., Jo

Re: Review Request 47978: VDF should use package-version for the os, not the release

> On June 1, 2016, 9:03 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/repository/VersionDefinitionXml.java, > > lines 113-114 > > > > > > m_ > > Jonathan Hurley wrote:

Re: Review Request 48121: YARN default configs are invalid

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48121/ --- (Updated Июнь 1, 2016, 1:16 п.п.) Review request for Ambari, Andrew Onischuk an

Re: Review Request 47978: VDF should use package-version for the os, not the release

> On June 1, 2016, 9:03 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/repository/VersionDefinitionXml.java, > > lines 113-114 > > > > > > m_ Ugh - didn't mean to make t

Re: Review Request 47978: VDF should use package-version for the os, not the release

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47978/#review135789 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 48088: Stack VDF should ignore unsupported OS

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48088/#review135788 --- Ship it! ambari-server/src/main/java/org/apache/ambari/server/

Review Request 48125: AMBARI-16984 Add changes for Ranger Service to get previous_jdbc_jar_name

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

Re: Review Request 48119: View Instance: Data Migration

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48119/#review135786 --- Ship it! Ship It! - Gaurav Nagar On June 1, 2016, 10:20 a.m.

Review Request 48124: Host checkboxes getting unselected when user clicks on search host button and few minor issues

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48124/ --- Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Don Bosco Durai

Re: Review Request 47880: AMBARI-16890 Updating Ambari configs changes for latest Ranger configs

> On May 26, 2016, 5:16 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/config-upgrade.xml, > > line 124 > > > > > > This config definition is not needed since it alread

Review Request 48123: Update default ambari view read timeout to a larger value

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

Review Request 48122: History tab takes long to populate when there is more entry in history table.

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

Re: Review Request 48063: Redundant info under config tab for falcon

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48063/ --- (Updated June 1, 2016, 11:27 a.m.) Review request for Ambari and Vitalyi Brodet

Review Request 48121: YARN default configs are invalid

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

Review Request 48119: View Instance: Data Migration

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

Re: Review Request 48089: AMBARI-16972 Strange behavior of Add Version functionality

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48089/#review135777 --- Ship it! Ship It! - Aleksandr Kovalenko On Май 31, 2016, 8:1

Re: Review Request 47586: Log Feeder Global exclusion default filter configuration is not effective in the log search indexing.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47586/#review135775 --- Ship it! Ship It! - Oliver Szabo On June 1, 2016, 6:20 a.m.,