Re: Review Request 48273: AMBARI-17054 : Configure Atlas Ranger Plugin

2016-06-08 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48273/#review136656 --- Ship it! Ship It! - Srimanth Gunturi On June 6, 2016, 3:26

Re: Review Request 48547: AMBARI-17164 Handle Ranger upgrade scenario in Kerberized env

2016-06-10 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48547/#review137079 --- Ship it! Ship It! - Srimanth Gunturi On June 10, 2016, 12

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

2016-06-02 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48125/#review136041 --- Ship it! Ship It! - Srimanth Gunturi On June 1, 2016, 12:59

Re: Review Request 47676: Spark History Server heap size is not exposed (History Server crashed with OOM)

2016-05-25 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47676/#review134867 --- Ship it! Ship It! - Srimanth Gunturi On May 25, 2016, 8:06

Re: Review Request 47847: AMBARI-16755 Add spark.driver.extraLibraryPath

2016-05-25 Thread Srimanth Gunturi
/configuration/spark-defaults.xml (line 154) <https://reviews.apache.org/r/47847/#comment199804> WebHCat and Hive use "/usr/hdp/${hdp.version}" when specifying paths. - Srimanth Gunturi On May 25, 2016, 7:37 p.m., We

Re: Review Request 47785: Ambari install of Atlas should use external HBase and Logsearch SOLR

2016-05-25 Thread Srimanth Gunturi
/package/scripts/params.py (line 143) <https://reviews.apache.org/r/47785/#comment199821> Just wanted to make sure if this always be true? - Srimanth Gunturi On May 25, 2016, 10:06 p.m., Tom Beerbower

Re: Review Request 48288: Microsoft-r install script should use Ambari libraries instead of direct shell calls

2016-06-06 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48288/#review136385 --- Ship it! Ship It! - Srimanth Gunturi On June 6, 2016, 5:30

Re: Review Request 47722: AMBARI-16813 Ranger Usersync config to support Group Based Search for LDAP Sync Source

2016-05-25 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47722/#review134919 --- Ship it! Ship It! - Srimanth Gunturi On May 25, 2016, 10:38

Re: Review Request 47885: AMBARI-16894: Default Ranger repos for some services are not getting created

2016-05-27 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47885/#review135279 --- Ship it! Ship It! - Srimanth Gunturi On May 27, 2016, 4:30

Re: Review Request 47961: Web Client Requests Handled By Jetty Should Not Be Blocked By JMX Property Providers

2016-05-27 Thread Srimanth Gunturi
hreads + 10-queue got 22 requests, the 2 overflown requests will not be executed till all 20 got executed. Optimization is to immediately submit the overflown requests, when we know of completed requests. - Srimanth Gunturi On May 27, 2016, 7:18 p.m., Jonathan Hurley

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

2016-05-27 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47428/#review135341 --- Ship it! Ship It! - Srimanth Gunturi On May 27, 2016, 10:42

Re: Review Request 47984: Make storm.topology.submission.notifier.plugin.class property optional

2016-05-31 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47984/#review135731 --- Ship it! Ship It! - Srimanth Gunturi On May 28, 2016, 5:15

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

2016-06-01 Thread Srimanth Gunturi
_ranger_plugin_xml.py". Same comment applies to other changes in this patch which use "driver_curl_source" - Srimanth Gunturi On June 1, 2016, 12:59 p.m., Mugdha Varadkar wrote: > > --- > This is an au

Re: Review Request 48507: AMBARI-17074: Expose Spark daemon memory in Spark2

2016-06-16 Thread Srimanth Gunturi
/package/scripts/params.py (line 66) <https://reviews.apache.org/r/48507/#comment203310> Is it ok if 'spark_daemon_memory' will be empty when the if() condition will be false? It looks like in that case the spark2-env will have 'export SPARK_DAEMON_MEMORY=m' - Srimanth Gunturi On

Re: Review Request 48507: AMBARI-17074: Expose Spark daemon memory in Spark2

2016-06-21 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48507/#review138929 --- Ship it! Ship It! - Srimanth Gunturi On June 21, 2016, 6:18

Re: Review Request 48827: AMBARI-17286 Service config links should use public host name

2016-06-17 Thread Srimanth Gunturi
/stack_advisor.py (line 778) <https://reviews.apache.org/r/48827/#comment203486> Would be more efficient to use 'server_host' variable instead of resolving fqdn from socket everytime. Similar comment to other calls. - Srimanth Gunturi On June 16, 2016, 11:43 p.m., Zhe (Joe) Wang

Re: Review Request 48827: AMBARI-17286 Service config links should use public host name

2016-06-17 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48827/#review138319 --- Ship it! Ship It! - Srimanth Gunturi On June 17, 2016, 9:08

Re: Review Request 48726: AMBARI-17247 : Populate audit to solr / hdfs properties for Atlas

2016-06-15 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48726/#review137869 --- Ship it! Ship It! - Srimanth Gunturi On June 15, 2016, 3:42

Re: Review Request 48507: AMBARI-17074: Expose Spark daemon memory in Spark2

2016-06-23 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48507/#review139301 --- Ship it! Ship It! - Srimanth Gunturi On June 22, 2016, 6:09

Re: Review Request 49193: AMBARI-16756 Update code for Ranger Service to use PID file for status and stop call

2016-06-27 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49193/#review139635 --- Ship it! Ship It! - Srimanth Gunturi On June 27, 2016, 11

Re: Review Request 44880: AMBARI-15436 Supportability: Create 'Reset UI' button in Ambari's experimental page

2016-03-19 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44880/#review124098 --- Ship it! Ship It! - Srimanth Gunturi On March 17, 2016, 8

Re: Review Request 45841: Restart LLAP action should be shown only when Hive Interactive Server is present in the cluster

2016-04-06 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45841/#review127508 --- Ship it! Ship It! - Srimanth Gunturi On April 6, 2016, 11

Re: Review Request 46105: capacity-scheduler config blob is not a part of recommendation API when Hive Interactive query is turned ON

2016-04-12 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46105/#review128492 --- Ship it! Ship It! - Srimanth Gunturi On April 12, 2016, 6

Re: Review Request 45705: HAWQ segment percentage alert critical threshold should be 25%

2016-04-04 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45705/#review127014 --- Ship it! Ship It! - Srimanth Gunturi On April 4, 2016, 9:48

Re: Review Request 46500: Regenerating keytabs on re-imaged hosts results in error during 'Creating Principals'

2016-04-21 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46500/#review129921 --- Ship it! Ship It! - Srimanth Gunturi On April 21, 2016, 5

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-04-22 Thread Srimanth Gunturi
> On April 22, 2016, 5:10 p.m., Srimanth Gunturi wrote: > > ambari-server/src/main/java/org/apache/ambari/server/api/services/stackadvisor/commands/StackAdvisorCommand.java, > > line 77 > > <https://reviews.apache.org/r/44210/diff/2/?file=1348336#file1348336line77>

Re: Review Request 47676: Spark History Server heap size is not exposed (History Server crashed with OOM)

2016-05-24 Thread Srimanth Gunturi
which is not possible with unit of 'G'. - Srimanth Gunturi On May 24, 2016, 9:23 p.m., Weiqing Yang wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: Review Request 47456: Takeover config merge should handle AMS hbase configs

2016-05-18 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47456/#review133796 --- Ship it! Ship It! - Srimanth Gunturi On May 18, 2016, 3:06

Re: Review Request 47288: AMBARI-16225. Storm Ambari view should work with Firefox 25

2016-05-18 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47288/#review133800 --- Ship it! Ship It! - Srimanth Gunturi On May 17, 2016, 5:51

Re: Review Request 47235: Takeover script on Paypal configs results in conflict message that look wrong

2016-05-11 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47235/#review132678 --- Ship it! Ship It! - Srimanth Gunturi On May 11, 2016, 1:54

Re: Review Request 47456: Takeover config merge should handle AMS hbase configs

2016-05-17 Thread Srimanth Gunturi
This file by default should have all the most common paths, and their correct config-types. - Srimanth Gunturi On May 17, 2016, 2:16 p.m., Andrew Onischuk wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request 47331: AMBARI-16649 : RegionServer and Nimbus start failed on secure cluster due to Ranger stack changes

2016-05-13 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47331/#review133105 --- Ship it! Ship It! - Srimanth Gunturi On May 13, 2016, 9:46

Re: Review Request 47014: Blueprint processor should create ConfigGroup even with only one host registered

2016-05-05 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47014/#review131871 --- Ship it! Ship It! - Srimanth Gunturi On May 5, 2016, 3:55

Re: Review Request 47048: Show background op window when Interactive query is enabled/disabled

2016-05-05 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47048/#review131975 --- Ship it! Ship It! - Srimanth Gunturi On May 6, 2016, 12:16

Re: Review Request 45713: AMBARI-15700: Stack advisor - Component layout recommendation needs support for avoiding hosts in maintenance mode

2016-04-18 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45713/#review129356 --- Ship it! Ship It! - Srimanth Gunturi On April 11, 2016, 5

Re: Review Request 46194: HDP 2.3.2.0: Changing hive tez container size (via ambari) should change tez.runtime.io.sort.mb as well

2016-04-14 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46194/#review128918 --- Ship it! Ship It! - Srimanth Gunturi On April 14, 2016, 11

Re: Review Request 49897: AMBARI-17617: Set default log level for Ranger KMS to INFO

2016-07-13 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49897/#review142174 --- Ship it! Ship It! - Srimanth Gunturi On July 11, 2016, 2:39

Re: Review Request 49989: AMBARI-17682 : For hive and hbase two properties are present policy.grantrevoke.auth.users & policy.grant.revoke.auth.users

2016-07-13 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49989/#review142175 --- Ship it! Ship It! - Srimanth Gunturi On July 13, 2016, 1:21

Re: Review Request 49908: Storm Ambari View should provide a config page & make calls to Storm Rest API

2016-07-13 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49908/#review142111 --- Ship it! Ship It! - Srimanth Gunturi On July 13, 2016, 4:12

Re: Review Request 49705: AMBARI-17586 : EU failed with errors during Kafka broker restart in Upgrade process

2016-07-07 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49705/#review141195 --- Ship it! Ship It! - Srimanth Gunturi On July 7, 2016, 7:17

Re: Review Request 49752: AMBARI-17000 Allow ranger admin credentials to be specified during cluster creation

2016-07-07 Thread Srimanth Gunturi
/package/scripts/ranger_admin.py (line 56) <https://reviews.apache.org/r/49752/#comment206624> Is changing password supported only during install? After install if this password changes, do we need to re-invoke this method? - Srimanth Gunturi On July 7, 2016, 12:54 p.m., Mugdha Va

Re: Review Request 49754: AMBARI-17607 Add localjecks support in ambari for Ranger and Ranger KMS services

2016-07-07 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49754/#review141217 --- Ship it! Ship It! - Srimanth Gunturi On July 7, 2016, 1:02