Re: Review Request 41397: AMBARI-14383 Add support for Ranger TagSync process as a component under RANGER

2016-03-31 Thread Gautam Borad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41397/ --- (Updated April 1, 2016, 5:59 a.m.) Review request for Ambari, Alejandro Fernand

Re: Review Request 41397: AMBARI-14383 Add support for Ranger TagSync process as a component under RANGER

2016-03-31 Thread Gautam Borad
> On March 31, 2016, 6:03 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_tagsync.py, > > line 33 > > > > > > Is this an optional component? >

Review Request 44982: AMBARI-15430: alert_group table has service info after service has been removed

2016-03-31 Thread Qin Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44982/ --- Review request for Ambari, Di Li and Jonathan Hurley. Bugs: AMBARI-15430 ht

Re: Review Request 45541: AMBARI-15648 Ranger KMS server start fails due to missing JCE policies on non-kerberos cluster

2016-03-31 Thread Mugdha Varadkar
> On March 31, 2016, 5:51 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/package/scripts/kms.py, > > line 423 > > > > > > Does this need to specify permiss

Re: Review Request 45566: HAWQ config should not allow multiple Master/Segment directories

2016-03-31 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45566/#review126532 --- ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_ad

Re: Review Request 45518: AMBARI-15638 : AMS Sum Calculation Incorrect

2016-03-31 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45518/#review126495 --- Ship it! Ship It! - Sumit Mohanty On March 31, 2016, 9:35 p.

Re: Review Request 45518: AMBARI-15638 : AMS Sum Calculation Incorrect

2016-03-31 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45518/#review126494 --- ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/a

Re: Review Request 45374: Use ">>" instead of ">" to write ambari-metrics-collector.out

2016-03-31 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45374/#review126492 --- Ship it! Ship It! - Aravindan Vijayan On March 30, 2016, 2:0

Re: Review Request 45548: Upgrading Kerberized JournalNode requires HDFS principal to perform 'role edits' task

2016-03-31 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45548/#review126487 --- Ship it! If jn_kinit_cmd is not used anywhere, lets remove it.

Re: Review Request 45568: AMBARI-15658. If there is a host with no components then host page spins

2016-03-31 Thread Xi Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45568/#review126482 --- Ship it! Ship It! - Xi Wang On March 31, 2016, 3:31 p.m., Ri

Review Request 45568: AMBARI-15658. If there is a host with no components then host page spins

2016-03-31 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45568/ --- Review request for Ambari and Xi Wang. Bugs: AMBARI-15658 https://issues.ap

Review Request 45566: HAWQ config should not allow multiple Master/Segment directories

2016-03-31 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45566/ --- Review request for Ambari, Andriy Babiichuk, Alexander Denissov, Alexandr Antone

Re: Review Request 45519: AMBARI-15637. BRANCH-2.2 If RU/EU is paused, services are restarted on the older version. EU is more complex since stopping services should use the original version.

2016-03-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45519/ --- (Updated March 31, 2016, 10 p.m.) Review request for Ambari, Dmytro Grinenko, D

Re: Review Request 45544: During cluster creation using Blueprints the cluster creation request has incorrect COMPLETED state instead of PENDING.

2016-03-31 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45544/#review126459 --- Ship it! Ship It! - Jayush Luniya On March 31, 2016, 2:28 p.

Re: Review Request 45544: During cluster creation using Blueprints the cluster creation request has incorrect COMPLETED state instead of PENDING.

2016-03-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45544/#review126458 --- Ship it! Ship It! - Alejandro Fernandez On March 31, 2016, 2

Re: Review Request 45518: AMBARI-15638 : AMS Sum Calculation Incorrect

2016-03-31 Thread Aravindan Vijayan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45518/ --- (Updated March 31, 2016, 9:35 p.m.) Review request for Ambari, Dmytro Sen, Sumi

Re: Review Request 45544: During cluster creation using Blueprints the cluster creation request has incorrect COMPLETED state instead of PENDING.

2016-03-31 Thread Sebastian Toader
> On March 31, 2016, 7:48 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java, > > line 512 > > > > > > Does this only affect bl

Re: Review Request 45560: AMBARI-15655: Remove remaining hdp specific logic from resource_management library

2016-03-31 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45560/ --- (Updated March 31, 2016, 9:16 p.m.) Review request for Ambari, Alejandro Fernan

Re: Review Request 45519: AMBARI-15637. BRANCH-2.2 If RU/EU is paused, services are restarted on the older version. EU is more complex since stopping services should use the original version.

2016-03-31 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45519/#review126439 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 45548: Upgrading Kerberized JournalNode requires HDFS principal to perform 'role edits' task

2016-03-31 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45548/#review126438 --- Maybe doc why HDFS is used instead of the JN credentials ... just

Re: Review Request 45548: Upgrading Kerberized JournalNode requires HDFS principal to perform 'role edits' task

2016-03-31 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45548/#review126437 --- Ship it! Ship It! - Jonathan Hurley On March 31, 2016, 12:35

Re: Review Request 45560: AMBARI-15655: Remove remaining hdp specific logic from resource_management library

2016-03-31 Thread Jayush Luniya
> On March 31, 2016, 8:27 p.m., Alejandro Fernandez wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/get_lzo_packages.py, > > line 41 > > > > > > Just to confirm, this is using unforma

Re: Review Request 45462: AMBARI-15604 Alert Definitions Repeat Tolerance Values Exposed In Web Client

2016-03-31 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45462/#review126432 --- Ship it! I'm normally not a web reviewer since I have limited w

Re: Review Request 45560: AMBARI-15655: Remove remaining hdp specific logic from resource_management library

2016-03-31 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45560/ --- (Updated March 31, 2016, 8:29 p.m.) Review request for Ambari, Alejandro Fernan

Re: Review Request 45560: AMBARI-15655: Remove remaining hdp specific logic from resource_management library

2016-03-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45560/#review126431 --- ambari-common/src/main/python/resource_management/libraries/funct

Re: Review Request 45561: Record and Expose Alert Occurrence Values

2016-03-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45561/#review126430 --- Ship it! Ship It! - Alejandro Fernandez On March 31, 2016, 8

Re: Review Request 45561: Record and Expose Alert Occurrence Values

2016-03-31 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45561/ --- (Updated March 31, 2016, 4:22 p.m.) Review request for Ambari, Dmitro Lisnichen

Re: Review Request 45560: AMBARI-15655: Remove remaining hdp specific logic from resource_management library

2016-03-31 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45560/ --- (Updated March 31, 2016, 8:21 p.m.) Review request for Ambari, Alejandro Fernan

Review Request 45561: Record and Expose Alert Occurrence Values

2016-03-31 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45561/ --- Review request for Ambari, Dmitro Lisnichenko, Jonathan Hurley, Nate Cole, and R

Review Request 45560: AMBARI-15655: Remove remaining hdp specific logic from resource_management library

2016-03-31 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45560/ --- Review request for Ambari, Alejandro Fernandez, Juanjo Marron, and Sumit Mohant

Re: Review Request 45486: AMBARI-15628 Ranger: update code for jdbc according to new logic

2016-03-31 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45486/#review126426 --- Ship it! Ship It! - Jayush Luniya On March 30, 2016, 2:20 p.

Re: Review Request 45470: AMBARI-15578: Stack Featurize Atlas Service

2016-03-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45470/#review126427 --- Ship it! Ship It! - Alejandro Fernandez On March 30, 2016, 5

Re: Review Request 45486: AMBARI-15628 Ranger: update code for jdbc according to new logic

2016-03-31 Thread Jayush Luniya
> On March 30, 2016, 5:43 p.m., Jayush Luniya wrote: > > - Can you add some more description in the Apache JIRA about change? > > - It looks like there is a dependent change that has been made in the past > > and the context is missing in the Apache JIRA > > Mugdha Varadkar wrote: > This R

Re: Review Request 45541: AMBARI-15648 Ranger KMS server start fails due to missing JCE policies on non-kerberos cluster

2016-03-31 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45541/#review126419 --- Ship it! Ship It! - Jayush Luniya On March 31, 2016, 1:46 p.

Re: Review Request 41397: AMBARI-14383 Add support for Ranger TagSync process as a component under RANGER

2016-03-31 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41397/#review126418 --- ambari-common/src/main/python/resource_management/libraries/funct

Re: Review Request 45519: AMBARI-15637. BRANCH-2.2 If RU/EU is paused, services are restarted on the older version. EU is more complex since stopping services should use the original version.

2016-03-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45519/ --- (Updated March 31, 2016, 7:54 p.m.) Review request for Ambari, Dmytro Grinenko,

Re: Review Request 45519: AMBARI-15637. BRANCH-2.2 If RU/EU is paused, services are restarted on the older version. EU is more complex since stopping services should use the original version.

2016-03-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45519/ --- (Updated March 31, 2016, 7:53 p.m.) Review request for Ambari, Dmytro Grinenko,

Re: Review Request 45554: ambari-server database check is failing due to EOF Error

2016-03-31 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45554/#review126414 --- Ship it! Ship It! - Jayush Luniya On March 31, 2016, 6:04 p.

Re: Review Request 45519: AMBARI-15637. BRANCH-2.2 If RU/EU is paused, services are restarted on the older version. EU is more complex since stopping services should use the original version.

2016-03-31 Thread Alejandro Fernandez
> On March 31, 2016, 5:44 a.m., Jayush Luniya wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java, > > line 1221 > > > > > > This essentially enforces non-rolling upgrade

Re: Review Request 45519: AMBARI-15637. BRANCH-2.2 If RU/EU is paused, services are restarted on the older version. EU is more complex since stopping services should use the original version.

2016-03-31 Thread Alejandro Fernandez
> On March 31, 2016, 1:57 p.m., Jonathan Hurley wrote: > > Something doesn't seem right with this Jira. We should always be using the > > desired version when restarting a service. This would prevent the situation > > to begin with. > > > > But let's say we do nothing; then the following fiel

Re: Review Request 45554: ambari-server database check is failing due to EOF Error

2016-03-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45554/#review126386 --- Ship it! Ship It! - Alejandro Fernandez On March 31, 2016, 6

Re: Review Request 45554: ambari-server database check is failing due to EOF Error

2016-03-31 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45554/#review126377 --- Ship it! Ship It! - Sumit Mohanty On March 31, 2016, 6:04 p.

Review Request 45554: ambari-server database check is failing due to EOF Error

2016-03-31 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45554/ --- Review request for Ambari, Andrew Onischuk, Dmytro Sen, and Sumit Mohanty. Bugs

Re: Review Request 41397: AMBARI-14383 Add support for Ranger TagSync process as a component under RANGER

2016-03-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41397/#review126375 --- ambari-server/src/main/resources/common-services/RANGER/0.4.0/pac

Re: Review Request 45540: Alerts Using the SKIPPED State Cause Stale Alert Notification

2016-03-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45540/#review126371 --- Ship it! Ship It! - Alejandro Fernandez On March 31, 2016, 1

Re: Review Request 45541: AMBARI-15648 Ranger KMS server start fails due to missing JCE policies on non-kerberos cluster

2016-03-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45541/#review126370 --- ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0

Re: Review Request 45544: During cluster creation using Blueprints the cluster creation request has incorrect COMPLETED state instead of PENDING.

2016-03-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45544/#review126368 --- ambari-server/src/main/java/org/apache/ambari/server/controller/i

Re: Review Request 45442: Orphaned Host Alerts Cause Stale Alert Notifications After Removing Hosts

2016-03-31 Thread Alejandro Fernandez
> On March 31, 2016, 1:13 a.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/events/listeners/alerts/AlertReceivedListener.java, > > line 403 > > > > > > This will acquire a re

Re: Review Request 45547: Sqoop client install

2016-03-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45547/#review126365 --- ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/

Re: Review Request 45548: Upgrading Kerberized JournalNode requires HDFS principal to perform 'role edits' task

2016-03-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45548/#review126363 --- Ship it! Ship It! - Alejandro Fernandez On March 31, 2016, 4

Re: Review Request 45054: AMBARI-15443:Make Host bulk command menu item list stack driven instead of a hardcoded list in UI code

2016-03-31 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45054/#review126362 --- Ship it! Ship It! - Alejandro Fernandez On March 31, 2016, 4

Re: Review Request 45549: Blueprint installation with Hive without Atlas fails

2016-03-31 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45549/ --- (Updated March 31, 2016, 4:50 p.m.) Review request for Ambari and Dmitro Lisnic

Re: Review Request 45054: AMBARI-15443:Make Host bulk command menu item list stack driven instead of a hardcoded list in UI code

2016-03-31 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45054/ --- (Updated March 31, 2016, 4:44 p.m.) Review request for Ambari, Alejandro Fernan

Re: Review Request 45549: Blueprint installation with Hive without Atlas fails

2016-03-31 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45549/#review126341 --- Fix it, then Ship it! The patch looks fine to me, but I'd recom

Re: Review Request 45549: Blueprint installation with Hive without Atlas fails

2016-03-31 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45549/#review126338 --- Ship it! Ship It! - Dmitro Lisnichenko On March 31, 2016, 7:

Review Request 45549: Blueprint installation with Hive without Atlas fails

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

Re: Review Request 45548: Upgrading Kerberized JournalNode requires HDFS principal to perform 'role edits' task

2016-03-31 Thread Mahadev Konar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45548/#review126331 --- Ship it! Ship It! - Mahadev Konar On March 31, 2016, 3:38 p.

Re: Review Request 45054: AMBARI-15443:Make Host bulk command menu item list stack driven instead of a hardcoded list in UI code

2016-03-31 Thread Di Li
> On March 31, 2016, 1:54 a.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/BulkCommandDefinition.java, > > line 70 > > > > > > May want to check these these are not null

Re: Review Request 45518: AMBARI-15638 : AMS Sum Calculation Incorrect

2016-03-31 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45518/#review126321 --- Ship it! Ship It! - Dmytro Sen On Март 31, 2016, 1:36 д.п.,

Re: Review Request 45545: Output short logs summary on start/stop failure

2016-03-31 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45545/ --- (Updated March 31, 2016, 3:43 p.m.) Review request for Ambari and Dmitro Lisnic

Re: Review Request 45545: Output short logs summary on start/stop failure

2016-03-31 Thread Myroslav Papirkovskyy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45545/#review126319 --- Ship it! Ship It! - Myroslav Papirkovskyy On Березень 31, 20

Review Request 45548: Upgrading Kerberized JournalNode requires HDFS principal to perform 'role edits' task

2016-03-31 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45548/ --- Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Jayush Luniya,

Re: Review Request 45518: AMBARI-15638 : AMS Sum Calculation Incorrect

2016-03-31 Thread Aravindan Vijayan
> On March 31, 2016, 10:36 a.m., Dmytro Sen wrote: > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/aggregators/TimelineMetricReadHelper.java, > > line 75 > >

Re: Review Request 45547: Sqoop client install

2016-03-31 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45547/#review126313 --- Ship it! Ship It! - Dmitro Lisnichenko On March 31, 2016, 6:

Review Request 45547: Sqoop client install

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

Re: Review Request 45545: Output short logs summary on start/stop failure

2016-03-31 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45545/#review126312 --- Ship it! Ship It! - Dmitro Lisnichenko On March 31, 2016, 5:

Re: Review Request 45545: Output short logs summary on start/stop failure

2016-03-31 Thread Myroslav Papirkovskyy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45545/#review126311 --- Ship it! Ship It! - Myroslav Papirkovskyy On Березень 31, 20

Re: Review Request 45442: Orphaned Host Alerts Cause Stale Alert Notifications After Removing Hosts

2016-03-31 Thread Jonathan Hurley
> On March 30, 2016, 9:13 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/events/listeners/alerts/AlertReceivedListener.java, > > line 403 > > > > > > This will acquire a re

Review Request 45545: Output short logs summary on start/stop failure

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

Re: Review Request 45545: Output short logs summary on start/stop failure

2016-03-31 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45545/ --- (Updated March 31, 2016, 2:50 p.m.) Review request for Ambari and Dmitro Lisnic

Re: Review Request 45544: During cluster creation using Blueprints the cluster creation request has incorrect COMPLETED state instead of PENDING.

2016-03-31 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45544/#review126308 --- Ship it! Ship It! - Oliver Szabo On March 31, 2016, 2:28 p.m

Re: Review Request 45544: During cluster creation using Blueprints the cluster creation request has incorrect COMPLETED state instead of PENDING.

2016-03-31 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45544/#review126307 --- Ship it! Ship It! - Daniel Gergely On márc. 31, 2016, 2:28 d

Re: Review Request 45544: During cluster creation using Blueprints the cluster creation request has incorrect COMPLETED state instead of PENDING.

2016-03-31 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45544/#review126306 --- Ship it! Ship It! - Sandor Magyari On March 31, 2016, 2:28 p

Review Request 45544: During cluster creation using Blueprints the cluster creation request has incorrect COMPLETED state instead of PENDING.

2016-03-31 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45544/ --- Review request for Ambari, Daniel Gergely, Oliver Szabo, and Sandor Magyari. Bu

Re: Review Request 45519: AMBARI-15637. BRANCH-2.2 If RU/EU is paused, services are restarted on the older version. EU is more complex since stopping services should use the original version.

2016-03-31 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45519/#review126305 --- Something doesn't seem right with this Jira. We should always be u

Re: Review Request 45538: Audit Log Code Cleanup & Safety

2016-03-31 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45538/ --- (Updated márc. 31, 2016, 1:51 du) Review request for Ambari, Jonathan Hurley, N

Review Request 45541: AMBARI-15648 Ranger KMS server start fails due to missing JCE policies on non-kerberos cluster

2016-03-31 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45541/ --- Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, Jonathan Hurley, a

Re: Review Request 45519: AMBARI-15637. BRANCH-2.2 If RU/EU is paused, services are restarted on the older version. EU is more complex since stopping services should use the original version.

2016-03-31 Thread Nate Cole
> On March 31, 2016, 1:44 a.m., Jayush Luniya wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java, > > line 1221 > > > > > > This essentially enforces non-rolling upgrade

Re: Review Request 45540: Alerts Using the SKIPPED State Cause Stale Alert Notification

2016-03-31 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45540/#review126304 --- Ship it! Ship It! - Nate Cole On March 31, 2016, 9:33 a.m.,

Review Request 45538: Audit Log Code Cleanup & Safety

2016-03-31 Thread Daniel Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45538/ --- Review request for Ambari, Jonathan Hurley, Nate Cole, and Sebastian Toader. Bu

Review Request 45540: Alerts Using the SKIPPED State Cause Stale Alert Notification

2016-03-31 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45540/ --- Review request for Ambari, Alejandro Fernandez, Dmitro Lisnichenko, and Nate Col

Re: Review Request 45486: AMBARI-15628 Ranger: update code for jdbc according to new logic

2016-03-31 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45486/#review126301 --- Ship it! Ship It! - Jonathan Hurley On March 30, 2016, 10:20

Re: Review Request 41397: AMBARI-14383 Add support for Ranger TagSync process as a component under RANGER

2016-03-31 Thread Gautam Borad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41397/ --- (Updated March 31, 2016, 11:38 a.m.) Review request for Ambari, Alejandro Ferna

Re: Review Request 41397: AMBARI-14383 Add support for Ranger TagSync process as a component under RANGER

2016-03-31 Thread Gautam Borad
> On Dec. 15, 2015, 8:41 p.m., Nate Cole wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/conf_select.py, > > lines 108-113 > > > > > > This will fail if conf-select doesn't support r

Re: Review Request 41397: AMBARI-14383 Add support for Ranger TagSync process as a component under RANGER

2016-03-31 Thread Gautam Borad
> On Dec. 15, 2015, 6:43 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/RANGER/0.4.0/package/scripts/ranger_tagsync.py, > > line 72 > > > > > > Is this already supported by hdp-

Re: Review Request 41397: AMBARI-14383 Add support for Ranger TagSync process as a component under RANGER

2016-03-31 Thread Gautam Borad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41397/ --- (Updated March 31, 2016, 11:16 a.m.) Review request for Ambari, Alejandro Ferna

Re: Review Request 45530: Dev Deploy: stack_advisor failed to recommend Hive Configurations (secured cluster)

2016-03-31 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45530/#review126294 --- Ship it! Ship It! - Dmytro Sen On Март 31, 2016, 11:12 д.п.,

Review Request 45530: Dev Deploy: stack_advisor failed to recommend Hive Configurations (secured cluster)

2016-03-31 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45530/ --- Review request for Ambari and Dmytro Sen. Bugs: AMBARI-15644 https://issues

Re: Review Request 45518: AMBARI-15638 : AMS Sum Calculation Incorrect

2016-03-31 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45518/#review126290 --- ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/a

Re: Review Request 45379: Introduce "Copy Path to clipboard" feature for Files browser view UI

2016-03-31 Thread Nitiraj Rathore
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45379/#review126287 --- Ship it! Ship It! - Nitiraj Rathore On March 30, 2016, 7:12

Re: Review Request 45486: AMBARI-15628 Ranger: update code for jdbc according to new logic

2016-03-31 Thread Mugdha Varadkar
> On March 30, 2016, 5:43 p.m., Jayush Luniya wrote: > > - Can you add some more description in the Apache JIRA about change? > > - It looks like there is a dependent change that has been made in the past > > and the context is missing in the Apache JIRA This Review is dependent on review requ