Re: Review Request 47211: AMBARI-16442 VDF UI edits and bugs: #2

2016-05-10 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47211/#review132593 --- Ship it! Ship It! - Jaimin Jetly On May 11, 2016, 1:11

Re: Review Request 47206: AMBARI-16441: RBAC UI Edits #6

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

Review Request 47206: AMBARI-16441: RBAC UI Edits #6

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

Re: Review Request 47009: Falcon server start fails

2016-05-10 Thread Tom Beerbower
> On May 9, 2016, 9:49 p.m., Venkat Ranganathan wrote: > > > > Venkat Ranganathan wrote: > We have committed a change as part of FALCON-1942 to add a environment > variable called FALCON_EXTRA_CLASS_PATH - this will be appended to the Falcon > server classpath. Please use it Great!

Re: Review Request 47202: Restarting Upgraded Component During a Paused Upgrade Downgrades Component To Old Version

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

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

2016-05-10 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46808/#review132558 ---

Re: Review Request 47156: More information for Standby sync alert

2016-05-10 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47156/#review132557 --- Ship it! Ship It! - Lav Jain On May 10, 2016, 10:22 p.m.,

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

2016-05-10 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46808/#review132555 ---

Re: Review Request 47156: More information for Standby sync alert

2016-05-10 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47156/ --- (Updated May 10, 2016, 10:22 p.m.) Review request for Ambari. Bugs:

Re: Review Request 47156: More information for Standby sync alert

2016-05-10 Thread jun aoki
> On May 10, 2016, 9:53 p.m., Lav Jain wrote: > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/package/alerts/alert_sync_status.py, > > line 93 > > > > > > Please try to use one SQL statement.

Re: Review Request 47156: More information for Standby sync alert

2016-05-10 Thread jun aoki
> On May 10, 2016, 7:10 a.m., Matt wrote: > > > > Matt wrote: > Please double check if any unit tests have to be updated. will do - jun --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 47156: More information for Standby sync alert

2016-05-10 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47156/ --- (Updated May 10, 2016, 9:54 p.m.) Review request for Ambari. Changes ---

Re: Review Request 47156: More information for Standby sync alert

2016-05-10 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47156/#review132547 ---

Re: Review Request 47156: More information for Standby sync alert

2016-05-10 Thread jun aoki
> On May 10, 2016, 7:10 a.m., Matt wrote: > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/package/alerts/alert_sync_status.py, > > line 68 > > > > > > Would be better to have **'HAWQSTANDBY is not in

Re: Review Request 47184: AMBARI-16435 : Change Namenode Total File operations widget to request rate metric instead of actual metric.

2016-05-10 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47184/#review132539 --- Ship it! Ship It! - Jaimin Jetly On May 10, 2016, 7:45

Re: Review Request 47190: AMBARI-16438. Modifications to llap config names and descriptions

2016-05-10 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47190/#review132533 --- Ship it! Ship It! - Sumit Mohanty On May 10, 2016, 9:19

Review Request 47190: AMBARI-16438. Modifications to llap config names and descriptions

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

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

2016-05-10 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47142/ --- (Updated May 10, 2016, 7:58 p.m.) Review request for Ambari, Dmytro Sen, Sumit

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

2016-05-10 Thread Ajit Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47142/#review132523 --- Ship it! Ship It! - Ajit Kumar On May 10, 2016, 7:56 p.m.,

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

2016-05-10 Thread Keta Patel
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46808/ --- (Updated May 10, 2016, 7:49 p.m.) Review request for Ambari, Di Li and Richard

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

2016-05-10 Thread Keta Patel
> On May 9, 2016, 9:42 p.m., Richard Zang wrote: > > File Attachment: AMBARI-15552-May-05.patch - AMBARI-15552-May-05.patch > > > > > > Coding style needs to be fixed(many places). Also, could you please > > attach the latest patch to JIRA

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

2016-05-10 Thread Keta Patel
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46808/ --- (Updated May 10, 2016, 7:48 p.m.) Review request for Ambari, Di Li and Richard

Review Request 47184: AMBARI-16435 : Change Namenode Total File operations widget to request rate metric instead of actual metric.

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

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

2016-05-10 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47142/ --- (Updated May 10, 2016, 7:39 p.m.) Review request for Ambari, Dmytro Sen, Sumit

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

2016-05-10 Thread Aravindan Vijayan
> On May 10, 2016, 7:01 p.m., Ajit Kumar wrote: > > ambari-metrics/ambari-metrics-timelineservice/src/test/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/TestPhoenixTransactSQL.java, > > line 561 > >

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

2016-05-10 Thread Aravindan Vijayan
> On May 10, 2016, 7:01 p.m., Ajit Kumar wrote: > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/query/ConditionBuilder.java, > > line 29 > >

Re: Review Request 47160: PXF should be collocated with NAMENODE even if NAMENODE is moved to another host

2016-05-10 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47160/#review132518 --- Do we have to Clean PXF from the previous NN node, if the node

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

2016-05-10 Thread Ajit Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47142/#review132515 ---

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

2016-05-10 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47142/#review132514 --- Ship it! Ship It! - Sid Wagle On May 10, 2016, 6:41 p.m.,

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

2016-05-10 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47142/ --- (Updated May 10, 2016, 6:41 p.m.) Review request for Ambari, Dmytro Sen, Sumit

Re: Review Request 47160: PXF should be collocated with NAMENODE even if NAMENODE is moved to another host

2016-05-10 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47160/#review132512 --- Ship it! Ship It! - Lav Jain On May 10, 2016, 7:26 a.m.,

Re: Review Request 47160: PXF should be collocated with NAMENODE even if NAMENODE is moved to another host

2016-05-10 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47160/#review132511 ---

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

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

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

2016-05-10 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47147/#review132490 --- Ship it! Ship It! - Sid Wagle On May 10, 2016, 12:41 a.m.,

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

2016-05-10 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47142/ --- (Updated May 10, 2016, 5:34 p.m.) Review request for Ambari, Dmytro Sen, Sumit

Re: Review Request 47140: Ambari Should Suspend Alerts Notifications During Upgrade

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

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

2016-05-10 Thread Sid Wagle
> On May 10, 2016, 12:27 a.m., Sid Wagle wrote: > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/HBaseTimelineMetricStore.java, > > line 308 > >

Re: Review Request 47160: PXF should be collocated with NAMENODE even if NAMENODE is moved to another host

2016-05-10 Thread Goutam Tadi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47160/#review132476 --- Ship it! Ship It! - Goutam Tadi On May 10, 2016, 7:26 a.m.,

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

2016-05-10 Thread Aravindan Vijayan
> On May 10, 2016, 12:27 a.m., Sid Wagle wrote: > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/HBaseTimelineMetricStore.java, > > line 308 > >

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

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

Re: Review Request 46944: AMBARI-16230: Sporadic errors when deploying the cluster

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

Re: Review Request 46944: AMBARI-16230: Sporadic errors when deploying the cluster

2016-05-10 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46944/ --- (Updated May 10, 2016, 4:19 p.m.) Review request for Ambari, Jonathan Hurley,

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

2016-05-10 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47026/#review132464 --- Fix it, then Ship it!

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

2016-05-10 Thread Jonathan Hurley
> On May 9, 2016, 3:39 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/view/RemoteAmbariCluster.java, > > lines 65-66 > > > > > > What was wrong with getName() ? this seems a

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

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

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

2016-05-10 Thread Gaurav Nagar
> On May 10, 2016, 3:44 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql, lines 791-797 > > > > > > I thought in the entities you had this as a Many-To-Many relationship

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

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

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

2016-05-10 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47023/ --- (Updated May 10, 2016, 3:29 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-10 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47027/ --- (Updated May 10, 2016, 3:25 p.m.) Review request for Ambari, DIPAYAN BHOWMICK,

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

2016-05-10 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47136/#review132440 --- Ship it! Ship It! - Nate Cole On May 9, 2016, 11:22 p.m.,

Re: Review Request 47009: Falcon server start fails

2016-05-10 Thread Tom Beerbower
> On May 9, 2016, 9:49 p.m., Venkat Ranganathan wrote: > > ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/configuration/falcon-env.xml, > > line 162 > > > > > > Don't we want the server classpath

Re: Review Request 46999: AMBARI-16261: LogFeeder logs are not showing up in LogSearch

2016-05-10 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46999/#review132427 --- Ship it! Ship It! - Robert Nettleton On May 10, 2016, 7:34

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

2016-05-10 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47136/#review132426 --- Ship it! Ship It! - Jonathan Hurley On May 9, 2016, 11:22

Re: Review Request 47163: AMBARI-16420: LogFeeder not using fully qualified domain name

2016-05-10 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47163/#review132425 --- Ship it! Ship It! - Robert Nettleton On May 10, 2016, 7:58

Re: Review Request 47140: Ambari Should Suspend Alerts Notifications During Upgrade

2016-05-10 Thread Jonathan Hurley
> On May 10, 2016, 5:01 a.m., Robert Levas wrote: > > ambari-server/src/main/java/org/apache/ambari/server/events/listeners/alerts/AlertStateChangedListener.java, > > line 192 > > > > > > "the an"? Don't fix

Re: Review Request 47130: Upgrade is blocked at 'Upgrade Options' wizard because of Service Checks requirement

2016-05-10 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47130/#review132422 ---

Re: Review Request 47130: Upgrade is blocked at 'Upgrade Options' wizard because of Service Checks requirement

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

Re: Review Request 47138: stackadvisor uses getHostsForSlaveComponent with wrong parameter name

2016-05-10 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47138/#review132419 --- Ship it! Ship It! - Tim Thorpe On May 9, 2016, 11:43 p.m.,

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

2016-05-10 Thread Gaurav Nagar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47026/ --- (Updated May 10, 2016, 11:19 a.m.) Review request for Ambari, DIPAYAN

Re: Review Request 46741: Config group UI enhancements

2016-05-10 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46741/#review132413 --- ambari-web/vendor/styles/bootstrap.css (line 3732)

Review Request 47179: Python UT fail

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

Re: Review Request 47163: AMBARI-16420: LogFeeder not using fully qualified domain name

2016-05-10 Thread Dharmesh Makwana
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47163/#review132407 --- Ship it! Ship It! - Dharmesh Makwana On May 10, 2016, 7:58

Re: Review Request 47176: Attempting to install on Amazon Linux, Agent fails to recognize OS

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

Review Request 47176: Attempting to install on Amazon Linux, Agent fails to recognize OS

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

Re: Review Request 47170: AMBARI-16360 RM fails to start after adding services in Kerb'd cluster

2016-05-10 Thread Balázs Bence Sári
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47170/ --- (Updated May 10, 2016, 10:04 a.m.) Review request for Ambari, Laszlo Puskas,

Re: Review Request 47011: Added top navbar on login page.

2016-05-10 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47011/#review132399 --- Ship it! Ship It! - Miklos Gergely On May 5, 2016, 12:46

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

2016-05-10 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47124/ --- (Updated May 10, 2016, 9:41 a.m.) Review request for Ambari, DIPAYAN BHOWMICK,

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

2016-05-10 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47124/ --- (Updated May 10, 2016, 9:39 a.m.) Review request for Ambari, DIPAYAN BHOWMICK,

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

2016-05-10 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47124/#review132396 ---

Re: Review Request 46999: AMBARI-16261: LogFeeder logs are not showing up in LogSearch

2016-05-10 Thread Don Bosco Durai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46999/ --- (Updated May 10, 2016, 7:34 a.m.) Review request for Ambari, Oliver Szabo and

Review Request 47160: PXF should be collocated with NAMENODE even if NAMENODE is moved to another host

2016-05-10 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47160/ --- Review request for Ambari, Alexander Denissov, bhuvnesh chaudhary, Goutam Tadi,

Re: Review Request 47156: More information for Standby sync alert

2016-05-10 Thread Matt
> On May 10, 2016, 12:10 a.m., Matt wrote: > > Please double check if any unit tests have to be updated. - Matt --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47156/#review132384