Re: Review Request 46770: Capacity Scheduler View: New UI - queues save integration

2016-05-09 Thread Akhil PB
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46770/ --- (Updated May 9, 2016, 6:37 a.m.) Review request for Ambari and Sreenath

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

2016-05-09 Thread Jeff Zhang
> On May 7, 2016, 7:31 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py, > > line 280 > > > > > > We should add this to HDP25StackAdvisor instead? > >

Review Request 47118: Fixed Typo in logfeeder property name

2016-05-09 Thread Hayat Behlim
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47118/ --- Review request for Ambari. Bugs: AMBARI-16304

Re: Review Request 47117: HiveServer interactive fails to start

2016-05-09 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47117/ --- (Updated máj. 9, 2016, 1:01 du) Review request for Ambari, Balázs Bence Sári,

Re: Review Request 47117: HiveServer interactive fails to start

2016-05-09 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47117/#review132266 --- Ship it! Ship It! - Laszlo Puskas On May 9, 2016, 1:01

Re: Review Request 47117: HiveServer interactive fails to start

2016-05-09 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47117/#review132267 --- Ship it! Ship It! - Sumit Mohanty On May 9, 2016, 1:01

Review Request 47119: LogFeeder: enable/disable log level filter

2016-05-09 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47119/ --- Review request for Ambari, Miklos Gergely, Robert Nettleton, and Sumit Mohanty.

Re: Review Request 47118: Fixed Typo in logfeeder property name

2016-05-09 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47118/#review132270 --- Ship it! Ship It! - Oliver Szabo On May 9, 2016, 12:53

Re: Review Request 47026: Views: Simplify View Configuration for Remote Ambari Managed Cluster. Ambari Database changes

2016-05-09 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47026/ --- (Updated May 9, 2016, 1:44 p.m.) Review request for Ambari, DIPAYAN BHOWMICK,

Re: Review Request 47027: Views: Simplify View Configuration for Remote Ambari Managed Cluster. Ambari Views changes

2016-05-09 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47027/ --- (Updated May 9, 2016, 1:48 p.m.) Review request for Ambari, DIPAYAN BHOWMICK,

Re: Review Request 47119: LogFeeder: enable/disable log level filter

2016-05-09 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47119/ --- (Updated May 9, 2016, 2 p.m.) Review request for Ambari, Miklos Gergely,

Re: Review Request 47117: HiveServer interactive fails to start

2016-05-09 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47117/ --- (Updated máj. 9, 2016, 2:18 du) Review request for Ambari, Balázs Bence Sári,

Re: Review Request 47119: LogFeeder: enable/disable log level filter

2016-05-09 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47119/#review132276 --- Fix it, then Ship it! Overal the patch looks fine to me, just

Re: Review Request 47119: LogFeeder: enable/disable log level filter

2016-05-09 Thread Oliver Szabo
> On May 9, 2016, 2:42 p.m., Robert Nettleton wrote: > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/params.py, > > line 275 > > > > > > This property doesn't appear to be

Re: Review Request 47119: LogFeeder: enable/disable log level filter

2016-05-09 Thread Robert Nettleton
> On May 9, 2016, 2:42 p.m., Robert Nettleton wrote: > > Overal the patch looks fine to me, just with a few minor issues below that > > need clarification. > > > > One additional question: How is the filter itself configured to a specific > > log level? Shouldn't there be some new stack

Re: Review Request 47119: LogFeeder: enable/disable log level filter

2016-05-09 Thread Oliver Szabo
> On May 9, 2016, 2:42 p.m., Robert Nettleton wrote: > > Overal the patch looks fine to me, just with a few minor issues below that > > need clarification. > > > > One additional question: How is the filter itself configured to a specific > > log level? Shouldn't there be some new stack

Re: Review Request 47117: HiveServer interactive fails to start

2016-05-09 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47117/#review132281 --- Ship it! - Sebastian Toader On May 9, 2016, 4:18 p.m.,

Re: Review Request 47060: AMBARI-16285 Ranger Tagsync stack changes to sync tags from Atlas

2016-05-09 Thread Jonathan Hurley
> On May 6, 2016, 10:29 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/params.py, > > line 230 > > > > > > Not using default? It's 100% guaranteed to

Re: Review Request 47026: Views: Simplify View Configuration for Remote Ambari Managed Cluster. Ambari Database changes

2016-05-09 Thread Jonathan Hurley
> On May 6, 2016, 10:12 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java, > > line 198 > > > > > > I don't think we need the word "ambari" in an

Re: Review Request 47074: AMBARI-16290 : Handle repository creation for Hive in Ranger for kerberised environments

2016-05-09 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47074/#review132286 --- There should be python tests to cover the new code paths. - Nate

Re: Review Request 47026: Views: Simplify View Configuration for Remote Ambari Managed Cluster. Ambari Database changes

2016-05-09 Thread Jonathan Hurley
> On May 6, 2016, 10:12 a.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java, > > line 198 > > > > > > I don't think we need the word "ambari" in an

Re: Review Request 47027: Views: Simplify View Configuration for Remote Ambari Managed Cluster. Ambari Views changes

2016-05-09 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47027/ --- (Updated May 9, 2016, 5:13 p.m.) Review request for Ambari, DIPAYAN BHOWMICK,

Re: Review Request 47023: Views: Simplify View Configuration for Remote Ambari Managed Cluster. Ambari Server Side changes

2016-05-09 Thread Gaurav Nagar
> On May 6, 2016, 5:53 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/RemoteAmbariClusterEntity.java, > > lines 83-86 > > > > > > Is this really a M2M

Re: Review Request 47027: Views: Simplify View Configuration for Remote Ambari Managed Cluster. Ambari Views changes

2016-05-09 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47027/#review132310 --- Ship it!

Re: Review Request 47023: Views: Simplify View Configuration for Remote Ambari Managed Cluster. Ambari Server Side changes

2016-05-09 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47023/#review132312 ---

Re: Review Request 47023: Views: Simplify View Configuration for Remote Ambari Managed Cluster. Ambari Server Side changes

2016-05-09 Thread Jonathan Hurley
> On May 6, 2016, 1:53 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/RemoteAmbariClusterEntity.java, > > lines 83-86 > > > > > > Is this really a M2M

Review Request 47134: HAWQ Password handling updated

2016-05-09 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47134/ --- Review request for Ambari, Alexander Denissov, jun aoki, Matt, and Oleksandr

Re: Review Request 47134: HAWQ Password handling updated

2016-05-09 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47134/#review132319 --- Ship it!

Review Request 47136: The 'krb5-conf' configuration is not available

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

Re: Review Request 46808: Role selection in List view of Manage Ambari page does not work correctly

2016-05-09 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46808/#review132335 --- File Attachment: AMBARI-15552-May-05.patch -

Re: Review Request 47009: Falcon server start fails

2016-05-09 Thread Venkat Ranganathan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47009/#review132337 ---

Re: Review Request 47134: HAWQ Password handling updated

2016-05-09 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47134/#review132339 --- Fix it, then Ship it!

Re: Review Request 47136: The 'krb5-conf' configuration is not available

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

Re: Review Request 47134: HAWQ Password handling updated

2016-05-09 Thread Alexander Denissov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47134/#review132347 --- Ship it! Ship It! - Alexander Denissov On May 9, 2016, 7:55

Review Request 47139: Ranger issues on summary page and config page

2016-05-09 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47139/ --- Review request for Ambari, Zhe (Joe) Wang, Richard Zang, and Srimanth Gunturi.

Re: Review Request 47139: Ranger issues on summary page and config page

2016-05-09 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47139/#review132348 --- Ship it! Ship It! - Zhe (Joe) Wang On May 9, 2016, 11:30

Re: Review Request 47139: Ranger issues on summary page and config page

2016-05-09 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47139/#review132349 --- Ship it! Ship It! - Richard Zang On May 9, 2016, 11:30

Review Request 47138: stackadvisor uses getHostsForSlaveComponent with wrong parameter name

2016-05-09 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47138/ --- Review request for Ambari. Bugs: AMBARI-16378

Review Request 47141: Stack advisor warning while looking for non-existing 'hive-interactive-site' when hive interactive is not installed/selected

2016-05-09 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47141/ --- Review request for Ambari, Jaimin Jetly and Sumit Mohanty. Bugs: AMBARI-16411

Re: Review Request 47141: Stack advisor warning while looking for non-existing 'hive-interactive-site' when hive interactive is not installed/selected

2016-05-09 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47141/#review132355 --- Ship it! Ship It! - Jaimin Jetly On May 9, 2016, 11:44

Re: Review Request 47085: Ambari agent changes to push llap metrics to AMS

2016-05-09 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47085/ --- (Updated May 10, 2016, 12:17 a.m.) Review request for Ambari, Aravindan

Re: Review Request 47142: AMBARI-16412 : Support TopN queries in AMS

2016-05-09 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47142/#review132356 --- Fix it, then Ship it!

Review Request 47147: AMBARI-16413: Ambari server does not start because of permission issues on symbolic links

2016-05-09 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47147/ --- Review request for Ambari, Andrew Onischuk, Mahadev Konar, and Sumit Mohanty.

Re: Review Request 47147: AMBARI-16413: Ambari server does not start because of permission issues on symbolic links

2016-05-09 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47147/ --- (Updated May 9, 2016, 5:41 p.m.) Review request for Ambari, Andrew Onischuk,

Review Request 47152: AMBARI-16414: Clients are not present in Hosts filter

2016-05-09 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47152/ --- Review request for Ambari, Jaimin Jetly and Zhe (Joe) Wang. Bugs: AMBARI-16414

Re: Review Request 47152: AMBARI-16414: Clients are not present in Hosts filter

2016-05-09 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47152/#review132364 --- Ship it! Ship It! - Zhe (Joe) Wang On May 10, 2016, 1:45

Re: Review Request 47136: The 'krb5-conf' configuration is not available

2016-05-09 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47136/ --- (Updated May 9, 2016, 11:21 p.m.) Review request for Ambari, Aravindan

Re: Review Request 47153: Storm - When kerberos enabled storm.thrift.transport is being set to wrong config

2016-05-09 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47153/#review132368 ---

Re: Review Request 47153: Storm - When kerberos enabled storm.thrift.transport is being set to wrong config

2016-05-09 Thread Sriharsha Chintalapani
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47153/ --- (Updated May 10, 2016, 3:59 a.m.) Review request for Ambari and Sumit Mohanty.

Re: Review Request 47153: Storm - When kerberos enabled storm.thrift.transport is being set to wrong config

2016-05-09 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47153/#review132369 --- Ship it! Ship It! - Sumit Mohanty On May 10, 2016, 3:59

Review Request 47156: More information for Standby sync alert

2016-05-09 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47156/ --- Review request for Ambari. Bugs: AMBARI-16417

Review Request 47124: Hive View : Upload Table- Error on uploading csv with 1 row

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