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

2016-05-27 Thread Sivaguru Chendamaraikannan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47984/ --- (Updated May 28, 2016, 5:15 a.m.) Review request for Ambari, Srimanth Gunturi

Re: Review Request 47996: AMBARI-16942. Take into account reading 'hive.tez.container.size', 'yarn.scheduler.minimum-allocation-mb', 'yarn.nodemanager.resource.memory-mb' & 'tez.am.resource.memory.mb'

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

Re: Review Request 47996: AMBARI-16942. Take into account reading 'hive.tez.container.size', 'yarn.scheduler.minimum-allocation-mb', 'yarn.nodemanager.resource.memory-mb' & 'tez.am.resource.memory.mb'

2016-05-27 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47996/ --- (Updated May 27, 2016, 11:43 p.m.) Review request for Ambari, Alejandro

Re: Review Request 47933: Add conditional constraints for Kerberos identities to control when they are created

2016-05-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47933/ --- (Updated May 27, 2016, 7:55 p.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 47933: Add conditional constraints for Kerberos identities to control when they are created

2016-05-27 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47933/#review135354 --- Ship it! Ship It! - Swapan Shridhar On May 27, 2016, 11:55

Re: Review Request 47993: VDF: support for selecting enabled + default Stacks

2016-05-27 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47993/#review135360 --- Ship it! Ship It! - Richard Zang On May 28, 2016, 12:13

Re: Review Request 47941: [AMBARI-16920] Spark2 thrift server can not started due to miss of spark-thrift-fairscheduler.xml

2016-05-27 Thread Jeff Zhang
> On May 27, 2016, 5:48 p.m., Alejandro Fernandez wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/copy_tarball.py, > > line 63 > > > > > > Why is the tarball source coming from /tmp?

Re: Review Request 47946: Yarn minimum container size calculation problem in stack advisor

2016-05-27 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47946/ --- (Updated máj. 27, 2016, 10:14 de) Review request for Ambari, Balázs Bence

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

2016-05-27 Thread Robert Levas
> On May 24, 2016, 4:41 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java, > > lines 190-194 > > > > > > Should special permissions

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

2016-05-27 Thread Robert Levas
> On May 25, 2016, 4:21 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java, > > lines 190-194 > > > > > > This is a little odd -

Review Request 47956: Automatically cleanup /var/run/ambari-server/stack-recommendations

2016-05-27 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47956/ --- Review request for Ambari and Srimanth Gunturi. Bugs: AMBARI-16929

Re: Review Request 47656: AMBARI-12885 - Dynamic stack extensions - install and upgrade support for custom services

2016-05-27 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47656/ --- (Updated May 27, 2016, 2:25 p.m.) Review request for Ambari, Alexander

Re: Review Request 47656: AMBARI-12885 - Dynamic stack extensions - install and upgrade support for custom services

2016-05-27 Thread Tim Thorpe
> On May 23, 2016, 10:49 a.m., Alexander Denissov wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/Extension.java, > > line 305 > > > > > > a lot of commented out code here I

Review Request 47954: AMBARI-16907 Ambari web UI does not auto-set all required properties when user toggles on the Yarn CPU isolation feature on the web UI

2016-05-27 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47954/ --- Review request for Ambari and Alejandro Fernandez. Bugs: AMBARI-16907

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

2016-05-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47783/ --- (Updated May 27, 2016, 9:52 a.m.) Review request for Ambari, Jonathan Hurley,

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

2016-05-27 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47730/ --- (Updated Май 27, 2016, 2:28 п.п.) Review request for Ambari, Aravindan

Review Request 47955: AmbariAMBARI-16906 Express upgrade: Oozie failed to start when user name and group are none-default values

2016-05-27 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47955/ --- Review request for Ambari and Alejandro Fernandez. Bugs: AMBARI-16906

Re: Review Request 47955: AMBARI-16906 Express upgrade: Oozie failed to start when user name and group are none-default values

2016-05-27 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47955/ --- (Updated May 27, 2016, 2:27 p.m.) Review request for Ambari and Alejandro

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

2016-05-27 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47730/#review135239 --- Ship it! Ship It! - Aravindan Vijayan On May 27, 2016, 2:28

Re: Review Request 47956: Automatically cleanup /var/run/ambari-server/stack-recommendations

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

Re: Review Request 47826: Insert Statement in Hive View is giving error in Kerberised cluster.

2016-05-27 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47826/#review135210 --- Ship it! Ship It! - Nitiraj Rathore On May 25, 2016, 12:39

Re: Review Request 47772: Upgrade existing capacity scheduler view instance after the Remote Ambari Cluster changes

2016-05-27 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47772/#review135212 --- Ship it! Ship It! - Nitiraj Rathore On May 24, 2016, 12:53

Re: Review Request 47950: Invalid syntax of Oracle DDL for cluster_handle in viewinstance

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

Re: Review Request 47947: AMBARI-16922 Remove LogSearch dependency for Ranger

2016-05-27 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47947/#review135227 --- Ship it! Ship It! - Oliver Szabo On May 27, 2016, 10:07

Re: Review Request 47953: Add ability to query all hosts using hostname=% through AMS API.

2016-05-27 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47953/ --- (Updated Май 27, 2016, 12:47 п.п.) Review request for Ambari, Aravindan

Re: Review Request 47825: View Config-Non remote user able to edit queue settings in remote capacity schedular view

2016-05-27 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47825/#review135211 --- Ship it! Ship It! - Nitiraj Rathore On May 25, 2016, 10:51

Re: Review Request 47950: Invalid syntax of Oracle DDL for cluster_handle in viewinstance

2016-05-27 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47950/#review135213 --- Ship it! Ship It! - Nitiraj Rathore On May 27, 2016, 10:51

Re: Review Request 47946: Yarn minimum container size calculation problem in stack advisor

2016-05-27 Thread Daniel Gergely
> On máj. 27, 2016, 11:05 de, Swapan Shridhar wrote: > > ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py, > > line 643 > > > > > > Should we do max here, or just pick configurations if >

Re: Review Request 47949: AMBARI-16923. Fix for getting the 'hive.llap.daemon.queue.name' config Property Attributes updated if there is a change in 'capacity-scheduler'.

2016-05-27 Thread Daniel Gergely
> On máj. 27, 2016, 12:09 du, Daniel Gergely wrote: > > Just a quiestion: if \n separated string is needed only for the UI, wouldnt it be better to handle it on the UI side? So UI can do the conversion, since it is not needed anywhere else. If it is also used somewhere else, then fine. -

Review Request 47953: Add ability to query all hosts using hostname=% through AMS API.

2016-05-27 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47953/ --- Review request for Ambari, Aravindan Vijayan, Myroslav Papirkovskyy, and Sid

Review Request 47947: AMBARI-16922 Remove LogSearch dependency for Ranger

2016-05-27 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47947/ --- Review request for Ambari, Don Bosco Durai, Gautam Borad, Mahadev Konar, Oliver

Review Request 47950: Invalid syntax of Oracle DDL for cluster_handle in viewinstance

2016-05-27 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47950/ --- Review request for Ambari, DIPAYAN BHOWMICK, Nitiraj Rathore, Pallav

Re: Review Request 47915: Oozie and Hive Server start fail during EU with missing DB class exception

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

Review Request 47946: Yarn minimum container size calculation problem in stack advisor

2016-05-27 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47946/ --- Review request for Ambari, Balázs Bence Sári, Laszlo Puskas, Oliver Szabo,

Re: Review Request 47856: AMBARI-16887 : [AMS / Grafana] Metrics are staying flat for 1 minutes, causing rate calculations to be 0

2016-05-27 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47856/#review135195 --- Ship it! Ship It! - Dmytro Sen On Май 26, 2016, 7:16 п.п.,

Review Request 47949: AMBARI-16923. Fix for getting the 'hive.llap.daemon.queue.name' config Property Attributes updated if there is a change in 'capacity-scheduler'.

2016-05-27 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47949/ --- Review request for Ambari, Alejandro Fernandez, Daniel Gergely, and Sumit

Re: Review Request 47946: Yarn minimum container size calculation problem in stack advisor

2016-05-27 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47946/#review135215 --- Good catch for requirement of looking into configurations also.

Re: Review Request 47956: Automatically cleanup /var/run/ambari-server/stack-recommendations

2016-05-27 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47956/ --- (Updated May 27, 2016, 3:07 p.m.) Review request for Ambari, Dmitro

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

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

Re: Review Request 47957: Add support for Ubuntu 16

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

Review Request 47960: Make it possible to debug ambari-agent in runtime to investigate memory leaks etc.

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

Re: Review Request 47960: Make it possible to debug ambari-agent in runtime to investigate memory leaks etc.

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

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

2016-05-27 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47961/ --- (Updated May 27, 2016, 12:26 p.m.) Review request for Ambari, Alejandro

Re: Review Request 47949: AMBARI-16923. Fix for getting the 'hive.llap.daemon.queue.name' config Property Attributes updated if there is a change in 'capacity-scheduler'.

2016-05-27 Thread Swapan Shridhar
> On May 27, 2016, 12:09 p.m., Daniel Gergely wrote: > > > > Daniel Gergely wrote: > Just a quiestion: if \n separated string is needed only for the UI, > wouldnt it be better to handle it on the UI side? So UI can do the > conversion, since it is not needed anywhere else. If it is also

Re: Review Request 47933: Add conditional constraints for Kerberos identities to control when they are created

2016-05-27 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47933/#review135255 ---

Re: Review Request 47933: Add conditional constraints for Kerberos identities to control when they are created

2016-05-27 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47933/#review135256 --- Ship it! Ship It! - Nate Cole On May 27, 2016, 6:01 a.m.,

Review Request 47964: clean up import * for ZOOKEEPER service

2016-05-27 Thread Juanjo Marron
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47964/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Matt. Bugs:

Re: Review Request 47953: Add ability to query all hosts using hostname=% through AMS API.

2016-05-27 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47953/#review135240 --- Ship it! Ship It! - Aravindan Vijayan On May 27, 2016,

Re: Review Request 47656: AMBARI-12885 - Dynamic stack extensions - install and upgrade support for custom services

2016-05-27 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47656/ --- (Updated May 27, 2016, 3:21 p.m.) Review request for Ambari, Alexander

Review Request 47957: Add support for Ubuntu 16

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

Re: Review Request 47957: Add support for Ubuntu 16

2016-05-27 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47957/ --- (Updated May 27, 2016, 4 p.m.) Review request for Ambari and Dmitro

Re: Review Request 47946: Yarn minimum container size calculation problem in stack advisor

2016-05-27 Thread Swapan Shridhar
> On May 27, 2016, 11:05 a.m., Swapan Shridhar wrote: > > ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py, > > line 643 > > > > > > Should we do max here, or just pick configurations if >

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

2016-05-27 Thread Nate Cole
> On May 24, 2016, 4:41 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java, > > lines 190-194 > > > > > > Should special permissions

Re: Review Request 47949: AMBARI-16923. Fix for getting the 'hive.llap.daemon.queue.name' config Property Attributes updated if there is a change in 'capacity-scheduler'.

2016-05-27 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47949/ --- (Updated May 27, 2016, 4:39 p.m.) Review request for Ambari, Alejandro

Review Request 47965: clean up import * for RANGER service

2016-05-27 Thread Juanjo Marron
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47965/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Matt. Bugs:

Re: Review Request 47915: Oozie and Hive Server start fail during EU with missing DB class exception

2016-05-27 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47915/ --- (Updated Травень 27, 2016, 4:47 після полудня) Review request for Ambari,

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

2016-05-27 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47961/#review135261 --- Ship it! Ship It! - Nate Cole On May 27, 2016, 12:26 p.m.,

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

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

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

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

Re: Review Request 47915: Oozie and Hive Server start fail during EU with missing DB class exception

2016-05-27 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47915/#review135259 --- Ship it! Ship It! - Andrew Onischuk On May 27, 2016, 4:47

Review Request 47967: Atlas server start failed after Ambari upgrade due to missing solrCloudCli.sh script

2016-05-27 Thread Tom Beerbower
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47967/ --- Review request for Ambari, John Speidel and Robert Levas. Bugs: AMBARI-16932

Re: Review Request 47933: Add conditional constraints for Kerberos identities to control when they are created

2016-05-27 Thread Robert Levas
> On May 27, 2016, 12:32 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/kerberos/eval/AndPredicate.java, > > lines 19-24 > > > > > > Seems like these are more generic than just

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

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

Re: Review Request 47949: AMBARI-16923. Fix for getting the 'hive.llap.daemon.queue.name' config Property Attributes updated if there is a change in 'capacity-scheduler'.

2016-05-27 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47949/#review135263 --- Ship it! Ship It! - Sumit Mohanty On May 27, 2016, 4:39

Re: Review Request 47949: AMBARI-16923. Fix for getting the 'hive.llap.daemon.queue.name' config Property Attributes updated if there is a change in 'capacity-scheduler'.

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

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

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

Re: Review Request 47965: clean up import * for RANGER service

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

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

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

Re: Review Request 47934: clean up import * for SLIDER service scripts in common-services

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

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

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

Re: Review Request 47962: clean up import * for STORM, TEZ and ZEPPELIN services

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

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

2016-05-27 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47961/#review135268 --- Ship it!

Re: Review Request 47746: Update Moment.js to latest stable version 2.13.0

2016-05-27 Thread Sangeeta Ravindran
> On May 24, 2016, 12:25 a.m., Jaimin Jetly wrote: > > Sangeeta Ravindran, Lets please add minified version of moment.js for > > 2.13.0 version. Hello Jaimin, I have attached a new patch with the minified version of moment.js 2.13.0. Can you please review? Thanks, Sangeeta - Sangeeta

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

2016-05-27 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47961/ --- (Updated May 27, 2016, 2:17 p.m.) Review request for Ambari, Alejandro

Re: Review Request 47947: AMBARI-16922 Remove LogSearch dependency for Ranger

2016-05-27 Thread Gautam Borad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47947/#review135285 --- Ship it! Ship It! - Gautam Borad On May 27, 2016, 10:07

Re: Review Request 47947: AMBARI-16922 Remove LogSearch dependency for Ranger

2016-05-27 Thread Don Bosco Durai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47947/#review135286 --- Ship it! Ship It! - Don Bosco Durai On May 27, 2016, 10:07

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

2016-05-27 Thread Robert Levas
> On May 27, 2016, 3:04 p.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java, > > line 52 > > > > > > Where is > >

Re: Review Request 47914: AMBARI-16910. Hive Server Interactive. Change the timeout to 120 secs for LLAP alert command.

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

Re: Review Request 47941: [AMBARI-16920] Spark2 thrift server can not started due to miss of spark-thrift-fairscheduler.xml

2016-05-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47941/#review135273 ---

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

2016-05-27 Thread Jonathan Hurley
> On May 27, 2016, 1:45 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/services/MetricsRetrievalService.java, > > lines 138-145 > > > > > > Doesn't seem like you need 2 of

Re: Review Request 47955: AMBARI-16906 Express upgrade: Oozie failed to start when user name and group are none-default values

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

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

2016-05-27 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47858/ --- (Updated May 27, 2016, 7:11 p.m.) Review request for Ambari, Alexander

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

2016-05-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47961/#review135291 --- Ship it! Ship It! - Robert Levas On May 27, 2016, 3:18

Re: Review Request 47922: clean up import * for KNOX service scripts in common-services

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

Re: Review Request 47884: Ambari install failure due to zeppelin service install bug

2016-05-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47884/#review135276 --- Ship it! Let me know if you need me to commit this on your

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

2016-05-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47961/#review135257 ---

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

2016-05-27 Thread Jonathan Hurley
> On May 27, 2016, 3:04 p.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/RestMetricsPropertyProvider.java, > > line 60 > > > > > > "returns is as a resource

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

2016-05-27 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47961/ --- (Updated May 27, 2016, 3:18 p.m.) Review request for Ambari, Alejandro

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

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

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 47933: Add conditional constraints for Kerberos identities to control when they are created

2016-05-27 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47933/#review135300 --- Ship it! Ship It! - Nate Cole On May 27, 2016, 3:58 p.m.,

Re: Review Request 47979: Deploy: UI: Hive_metastore not started

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

Re: Review Request 47967: Atlas server start failed after Ambari upgrade due to missing solrCloudCli.sh script

2016-05-27 Thread Tom Beerbower
> On May 27, 2016, 8:08 p.m., Robert Levas wrote: > > Its not clear how the patch helps to prevent: > > > > ``` > > Traceback (most recent call last): > > File > > "/var/lib/ambari-agent/cache/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata_server.py", > > line 165, in > >

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

2016-05-27 Thread Sivaguru Chendamaraikannan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47984/ --- Review request for Ambari, Srimanth Gunturi and Tom Beerbower. Bugs:

Re: Review Request 47967: Atlas server start failed after Ambari upgrade due to missing solrCloudCli.sh script

2016-05-27 Thread Robert Levas
> On May 27, 2016, 4:08 p.m., Robert Levas wrote: > > Its not clear how the patch helps to prevent: > > > > ``` > > Traceback (most recent call last): > > File > > "/var/lib/ambari-agent/cache/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata_server.py", > > line 165, in > >

Re: Review Request 47967: Atlas server start failed after Ambari upgrade due to missing solrCloudCli.sh script

2016-05-27 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47967/#review135312 --- Ship it! Ship It! - Robert Levas On May 27, 2016, 1 p.m.,

Review Request 47985: Fix for AMBARI-16939, upgrade from 2.2.0 to 2.4.0 on Ubuntu 14.04 / MySQL.

2016-05-27 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47985/ --- Review request for Ambari, Jonathan Hurley, Laszlo Puskas, Nate Cole, Oliver

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

2016-05-27 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47961/ --- (Updated May 27, 2016, 5:20 p.m.) Review request for Ambari, Alejandro

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

2016-05-27 Thread Jonathan Hurley
> On May 27, 2016, 4:06 p.m., Srimanth Gunturi wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/utilities/BufferedThreadPoolExecutorCompletionService.java, > > line 107 > > > > > > Small

Re: Review Request 47979: Deploy: UI: Hive_metastore not started

2016-05-27 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47979/#review135319 --- Ship it! Should add tests covering both hive/hive2. - Nate

  1   2   >