Re: Review Request 47344: Log count text on Log level select boxes in Service Logs Tab not getting updated while checking and unchecking

2016-05-24 Thread Don Bosco Durai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47344/#review134701 --- Ship it! Ship It! - Don Bosco Durai On May 13, 2016, 12:01

Re: Review Request 47802: expose vm.overcommit_ratio on hawq-sysctl-env

2016-05-24 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47802/#review134692 ---

Re: Review Request 47773: one very careless mistake for ambari agent cancel logic

2016-05-24 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47773/#review134694 --- Ship it! Ship It! - Sumit Mohanty On May 25, 2016, 2:18

Re: Review Request 47811: AMBARI-16862 : [Grafana] Rename "HBase - Performance" dashboard to "HBase - RegionServers"

2016-05-24 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47811/#review134691 --- Just wondering if we modified title to originalTitle in this code

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

2016-05-24 Thread Jianfeng (Jeff) Zhang
+ Sean Roberts who may able to help Hi Jayush & Sean, I also ask the question on ambari mail list, but still could not make progress, not sure how to verify that, do you have any suggestion on that ? Very appreciated.

Review Request 47811: AMBARI-16862 : [Grafana] Rename "HBase - Performance" dashboard to "HBase - RegionServers"

2016-05-24 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47811/ --- Review request for Ambari, Dmytro Sen, Prajwal Rao, Sumit Mohanty, Sid Wagle,

Re: Review Request 47790: Failed to install packages for HDP 2.4 and 2.5

2016-05-24 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47790/#review134684 --- Fix it, then Ship it! Just the one comment really about the

Re: Review Request 47773: one very careless mistake for ambari agent cancel logic

2016-05-24 Thread chen jian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47773/ --- (Updated 五月 25, 2016, 2:18 a.m.) Review request for Ambari, Alexandr

Re: Review Request 47790: Failed to install packages for HDP 2.4 and 2.5

2016-05-24 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47790/ --- (Updated May 24, 2016, 10:07 p.m.) Review request for Ambari, Alejandro

Re: Review Request 47605: AMBARI-16446 Ranger audit properties should be configured to use Ambari installed Solr (LogSearch)

2016-05-24 Thread Don Bosco Durai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47605/#review134681 --- Ship it! Ship It! - Don Bosco Durai On May 24, 2016, 11:58

Re: Review Request 47773: one very careless mistake for ambari agent cancel logic

2016-05-24 Thread chen jian
> On 五月 24, 2016, 1:23 p.m., Tom Beerbower wrote: > > The fix looks okay to me. How did you test it? Could you please complete > > the "testing done" section of the review? that's done, thanks. That is my first time commit for Ambari, Could you tell me what to do to close my jira issue.

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

2016-05-24 Thread Srimanth Gunturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47676/#review134671 ---

Re: Review Request 47802: expose vm.overcommit_ratio on hawq-sysctl-env

2016-05-24 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47802/#review134670 --- Ship it! Ship It! - Lav Jain On May 25, 2016, 12:17 a.m.,

Re: Review Request 47800: AMBARI-16860 : Disable HBase per user and per table metrics by default

2016-05-24 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47800/#review134668 --- Ship it! Ship It! - Sumit Mohanty On May 24, 2016, 11:40

Review Request 47802: expose vm.overcommit_ratio on hawq-sysctl-env

2016-05-24 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47802/ --- Review request for Ambari, Alexander Denissov, Lav Jain, and Matt. Bugs:

Re: Review Request 47773: one very careless mistake for ambari agent cancel logic

2016-05-24 Thread chen jian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47773/ --- (Updated 五月 25, 2016, 12:10 a.m.) Review request for Ambari, Alexandr

Review Request 47800: AMBARI-16860 : Disable HBase per user and per table metrics by default

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

Re: Review Request 47781: clean up import * for PIG service scripts in common-services

2016-05-24 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47781/#review134646 --- Ship it! Ship It! - Matt On May 24, 2016, 9:39 a.m., Juanjo

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

2016-05-24 Thread Weiqing Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47676/ --- (Updated May 24, 2016, 9:23 p.m.) Review request for Ambari, Sumit Mohanty and

Re: Review Request 47018: [preview] "ambari-server upgrade" shouldn't automatically add stack configs

2016-05-24 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47018/#review134634 --- ambari-server/src/main/resources/property-schema.xsd (lines 1 -

Re: Review Request 47018: [preview] "ambari-server upgrade" shouldn't automatically add stack configs

2016-05-24 Thread Nate Cole
> On May 19, 2016, 10:20 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/stack/StackManager.java, > > line 298 > > > > > > Make a constant or read it from XML (if you define XSD

Re: Review Request 47018: [preview] "ambari-server upgrade" shouldn't automatically add stack configs

2016-05-24 Thread Nate Cole
> On May 19, 2016, 10:20 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/stack/StackManager.java, > > line 298 > > > > > > Make a constant or read it from XML (if you define XSD

Re: Review Request 47783: Cluster operator and cluster admin not allowed to install ambari agent

2016-05-24 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47783/#review134630 --- Fix it, then Ship it!

Review Request 47790: Failed to install packages for HDP 2.4 and 2.5

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

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

2016-05-24 Thread Tom Beerbower
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47785/ --- Review request for Ambari. Bugs: AMBARI-16853 and ATLAS-823

Re: Review Request 47769: Need ability to ignoreBadMounts on a large cluster with bad disks vs workaround of tens of Config Groups

2016-05-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47769/#review134611 --- Fix it, then Ship it!

Review Request 47783: Cluster operator and cluster admin not allowed to install ambari agent

2016-05-24 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47783/ --- Review request for Ambari, Jonathan Hurley, Myroslav Papirkovskyy, and Nate

Re: Review Request 47729: MAPREDUCE2_SERVICE_CHECK failed on secured cluster

2016-05-24 Thread Mahadev Konar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47729/#review134610 --- Ship it! Ship It! - Mahadev Konar On May 23, 2016, 9:28

Re: Review Request 47596: HiveServer interactive - incorrect default memory value

2016-05-24 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47596/#review134606 --- Ship it! Ship It! - Swapan Shridhar On May 23, 2016, 8:20

Re: Review Request 47774: AMBARI-16843. Auto logout zeppelin server in case of session inactivity.

2016-05-24 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47774/#review134605 --- Ship it! Ship It! - Alejandro Fernandez On May 24, 2016,

Re: Review Request 47417: Replace '*' to hdp version in lzo packages

2016-05-24 Thread Myroslav Papirkovskyy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47417/#review134597 --- Ship it! Ship It! - Myroslav Papirkovskyy On Травень 24,

Re: Review Request 47018: [preview] "ambari-server upgrade" shouldn't automatically add stack configs

2016-05-24 Thread Dmitro Lisnichenko
> On May 19, 2016, 5:20 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/stack/StackManager.java, > > line 298 > > > > > > Make a constant or read it from XML (if you define XSD

Re: Review Request 47018: [preview] "ambari-server upgrade" shouldn't automatically add stack configs

2016-05-24 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47018/ --- (Updated May 24, 2016, 7:11 p.m.) Review request for Ambari, Alejandro

Re: Review Request 47730: Improve TimelineMetricsCache eviction/flush logic using a cache library

2016-05-24 Thread Dmytro Sen
> On Май 23, 2016, 3:59 п.п., Sid Wagle wrote: > > ambari-metrics/ambari-metrics-common/pom.xml, line 78 > > > > > > This will collide due to Hadoop dependency in version 14.0.1, I faced a > > problem here before.

Review Request 47774: AMBARI-16843. Auto logout zeppelin server in case of session inactivity.

2016-05-24 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47774/ --- Review request for Ambari, Alejandro Fernandez, DIPAYAN BHOWMICK, Jayush

Re: Review Request 47773: one very careless mistake for ambari agent cancel logic

2016-05-24 Thread Tom Beerbower
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47773/#review134576 --- Ship it! The fix looks okay to me. How did you test it?

Re: Review Request 47766: Host name resolution for dfs_ha_initial_namenode_active and dfs_ha_initial_namenode_standby properties.

2016-05-24 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47766/ --- (Updated May 24, 2016, 1:29 p.m.) Review request for Ambari, Daniel Gergely,

Re: Review Request 47774: AMBARI-16843. Auto logout zeppelin server in case of session inactivity.

2016-05-24 Thread Prabhjyot Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47774/ --- (Updated May 24, 2016, 1:27 p.m.) Review request for Ambari, Alejandro

Re: Review Request 47738: [Hive View] Non-visual explain gives a bunch of strangely clickable links

2016-05-24 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47738/#review134574 ---

Re: Review Request 47766: Host name resolution for dfs_ha_initial_namenode_active and dfs_ha_initial_namenode_standby properties.

2016-05-24 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47766/ --- (Updated May 24, 2016, 12:19 p.m.) Review request for Ambari, Daniel Gergely,

Re: Review Request 47770: yarn statrtup is failing in HA cluster in wireencypted cluster

2016-05-24 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47770/#review134565 --- Ship it! Ship It! - Dmitro Lisnichenko On May 24, 2016,

Re: Review Request 47768: Storm REST API alert is displayed and is always None

2016-05-24 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47768/ --- (Updated May 24, 2016, 11:34 a.m.) Review request for Ambari, Jonathan Hurley

Re: Review Request 47769: Need ability to ignoreBadMounts on a large cluster with bad disks vs workaround of tens of Config Groups

2016-05-24 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47769/ --- (Updated May 24, 2016, 11:29 a.m.) Review request for Ambari, Alejandro

Review Request 47769: Need ability to ignoreBadMounts on a large cluster with bad disks vs workaround of tens of Config Groups

2016-05-24 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47769/ --- Review request for Ambari and Alejandro Fernandez. Bugs: AMBARI-16839

Re: Review Request 47768: Storm REST API alert is displayed and is always None

2016-05-24 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47768/#review134560 --- Ship it! Ship It! - Vitalyi Brodetskyi On Травень 24, 2016,

Re: Review Request 47757: Desired Configuration Cache Expiration Caused 10, 000's of Database Hits In Large Deployments

2016-05-24 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47757/#review134559 --- Ship it! Ship It! - Nate Cole On May 23, 2016, 10:38 p.m.,

Re: Review Request 47605: AMBARI-16446 Ranger audit properties should be configured to use Ambari installed Solr (LogSearch)

2016-05-24 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47605/ --- (Updated May 24, 2016, 9:57 a.m.) Review request for Ambari, Alejandro

Re: Review Request 47764: WEBHDFS PUT fails on Amazon Linux

2016-05-24 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47764/#review134556 --- Ship it! Ship It! - Dmitro Lisnichenko On May 24, 2016,

Review Request 47764: WEBHDFS PUT fails on Amazon Linux

2016-05-24 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47764/ --- Review request for Ambari and Dmitro Lisnichenko. Bugs: AMBARI-16835

Re: Review Request 47696: Upload Table- Tries to upload table, when no file is mentioned

2016-05-24 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47696/#review134555 --- Ship it! Ship It! - Pallav Kulshreshtha On May 24, 2016,

Re: Review Request 47730: Improve TimelineMetricsCache eviction/flush logic using a cache library

2016-05-24 Thread Dmytro Sen
> On Май 23, 2016, 3:59 п.п., Sid Wagle wrote: > > ambari-metrics/ambari-metrics-common/pom.xml, line 78 > > > > > > This will collide due to Hadoop dependency in version 14.0.1, I faced a > > problem here before.

Re: Review Request 47696: Upload Table- Tries to upload table, when no file is mentioned

2016-05-24 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47696/#review134551 --- Ship it! Ship It! - DIPAYAN BHOWMICK On May 24, 2016, 9:09

Re: Review Request 47592: AMBARI-16772 Zeppelin restart with Ambari fails - according to Ambari

2016-05-24 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47592/#review134550 --- Ship it! Ship It! - Rohit Choudhary On May 19, 2016, 2:06

Re: Review Request 47611: AMBARI-16779 Zeppelin Ambari stack does not handle log4j configuration

2016-05-24 Thread Rohit Choudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47611/#review134549 --- Ship it! Ship It! - Rohit Choudhary On May 19, 2016, 6:11

Re: Review Request 47763: Any start command fails if AMS is installed.

2016-05-24 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47763/#review134548 --- Ship it! Ship It! - Dmitro Lisnichenko On May 24, 2016,

Review Request 47763: Any start command fails if AMS is installed.

2016-05-24 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47763/ --- Review request for Ambari and Sid Wagle. Bugs: AMBARI-16834

Re: Review Request 47596: HiveServer interactive - incorrect default memory value

2016-05-24 Thread Daniel Gergely
> On máj. 24, 2016, 4:03 de, Swapan Shridhar wrote: > > ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py, > > line 221 > > > > > > Can you explain how moving this line up helps ? Thx. I

Re: Review Request 47726: Blueprint Export does not replace hive.llap.zk.sm.connectionString

2016-05-24 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47726/#review134546 --- Ship it! Ship It! - Oliver Szabo On May 24, 2016, 8:55

Re: Review Request 47726: Blueprint Export does not replace hive.llap.zk.sm.connectionString

2016-05-24 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47726/ --- (Updated máj. 24, 2016, 8:55 de) Review request for Ambari, Balázs Bence Sári,

Re: Review Request 47726: Blueprint Export does not replace hive.llap.zk.sm.connectionString

2016-05-24 Thread Daniel Gergely
> On máj. 23, 2016, 2:59 du, Robert Nettleton wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java, > > line 2395 > > > > > > +1 for Sebastian's

Re: Review Request 47721: Editing local cluster name breaks all view instances

2016-05-24 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47721/#review134544 --- Ship it! Ship It! - Nitiraj Rathore On May 23, 2016, 10:21

Re: Review Request 47721: Editing local cluster name breaks all view instances

2016-05-24 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47721/#review134537 --- Ship it! Ship It! - DIPAYAN BHOWMICK On May 23, 2016, 10:21

Re: Review Request 47721: Editing local cluster name breaks all view instances

2016-05-24 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47721/#review134362 ---

Re: Review Request 47643: Capacity Scheduler View - Edit and validate queue name

2016-05-24 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47643/#review134530 --- Ship it! Ship It! - Pallav Kulshreshtha On May 23, 2016,

Re: Review Request 47644: All created views disappear after upgrade to 2.4.0.0

2016-05-24 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47644/#review134528 --- Ship it! Ship It! - DIPAYAN BHOWMICK On May 20, 2016, 10:15

Re: Review Request 47467: Hive View does not work with multi-bytes characters

2016-05-24 Thread DIPAYAN BHOWMICK
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47467/#review134527 --- Ship it! Ship It! - DIPAYAN BHOWMICK On May 17, 2016, 3:27