[jira] [Commented] (YARN-5516) Add REST API for periodicity

2017-10-26 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5516?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16220921#comment-16220921
 ] 

Sean Po commented on YARN-5516:
---

The following tests are failing but are tracked, known issues:.
hadoop.yarn.server.resourcemanager.scheduler.fair.TestFSAppStarvation tracked 
by YARN-6747
org.apache.hadoop.yarn.server.resourcemanager.TestSubmitApplicationWithRMHA 
tracked by YARN-7080

> Add REST API for periodicity
> 
>
> Key: YARN-5516
> URL: https://issues.apache.org/jira/browse/YARN-5516
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Sangeetha Abdu Jyothi
>Assignee: Sean Po
> Attachments: YARN-5516.v001.patch, YARN-5516.v002.patch, 
> YARN-5516.v003.patch, YARN-5516.v004.patch, YARN-5516.v005.patch, 
> YARN-5516.v006.patch
>
>
> YARN-5516 changing REST API of the reservation system to support periodicity. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5516) Add REST API for periodicity

2017-10-25 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5516?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5516:
--
Attachment: YARN-5516.v006.patch

Thanks [~subru] for the comments, the latest patch changes the dates in the 
InMemoryPlan tests to reference 2050 instead of 2017. 

> Add REST API for periodicity
> 
>
> Key: YARN-5516
> URL: https://issues.apache.org/jira/browse/YARN-5516
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Sangeetha Abdu Jyothi
>Assignee: Sean Po
> Attachments: YARN-5516.v001.patch, YARN-5516.v002.patch, 
> YARN-5516.v003.patch, YARN-5516.v004.patch, YARN-5516.v005.patch, 
> YARN-5516.v006.patch
>
>
> YARN-5516 changing REST API of the reservation system to support periodicity. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5516) Add REST API for periodicity

2017-10-25 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5516?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5516:
--
Attachment: YARN-5516.v005.patch

The following tests are failing but are also tracked.
hadoop.yarn.server.resourcemanager.scheduler.fair.TestFSAppStarvation tracked 
by YARN-6747
org.apache.hadoop.yarn.server.resourcemanager.TestSubmitApplicationWithRMHA 
tracked by YARN-7080

The following tests do not appear to be related to my changes.
hadoop.yarn.server.resourcemanager.reservation.TestCapacityOverTimePolicy
org.apache.hadoop.yarn.server.resourcemanager.TestReservationSystemWithRMHA

The latest patch fixes the bug in the InMemoryPlan I referenced above, and also 
adds a test with regards to the bug.

> Add REST API for periodicity
> 
>
> Key: YARN-5516
> URL: https://issues.apache.org/jira/browse/YARN-5516
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Sangeetha Abdu Jyothi
>Assignee: Sean Po
> Attachments: YARN-5516.v001.patch, YARN-5516.v002.patch, 
> YARN-5516.v003.patch, YARN-5516.v004.patch, YARN-5516.v005.patch
>
>
> YARN-5516 changing REST API of the reservation system to support periodicity. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5516) Add REST API for periodicity

2017-10-25 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5516?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16218925#comment-16218925
 ] 

Sean Po commented on YARN-5516:
---

There is a bug in the code in line 639 of InMemoryPlan, where the duration can 
potentially be negative. I have added a fix along with a test to catch this 
error. I will update a patch when the Yetus run completes for the previous 
patch in case there is any action required as a result of a failed Yetus run.

> Add REST API for periodicity
> 
>
> Key: YARN-5516
> URL: https://issues.apache.org/jira/browse/YARN-5516
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Sangeetha Abdu Jyothi
>Assignee: Sean Po
> Attachments: YARN-5516.v001.patch, YARN-5516.v002.patch, 
> YARN-5516.v003.patch, YARN-5516.v004.patch
>
>
> YARN-5516 changing REST API of the reservation system to support periodicity. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5516) Add REST API for periodicity

2017-10-25 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5516?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5516:
--
Attachment: YARN-5516.v004.patch

TestIncreaseAllocationExpirer tests fail intermittently and is tracked by 
YARN-7378. TestOpportunisticContainerAllocatorAMService also fails 
intermittently and is tracked by YARN-6841.

The checkstyle issues above will be fixed in YARN-5516.v004.patch.

> Add REST API for periodicity
> 
>
> Key: YARN-5516
> URL: https://issues.apache.org/jira/browse/YARN-5516
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Sangeetha Abdu Jyothi
>Assignee: Sean Po
> Attachments: YARN-5516.v001.patch, YARN-5516.v002.patch, 
> YARN-5516.v003.patch, YARN-5516.v004.patch
>
>
> YARN-5516 changing REST API of the reservation system to support periodicity. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5516) Add REST API for periodicity

2017-10-25 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5516?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5516:
--
Attachment: YARN-5516.v003.patch

Thanks [~subru] for the review. I added a change for all your comments. While 
adding the test for the RmWebServices, I found that there was a bug in the 
getReservations API of the InMemoryPlan. That bug has been fixed, and test 
cases were also created to catch future bugs in this area. 

> Add REST API for periodicity
> 
>
> Key: YARN-5516
> URL: https://issues.apache.org/jira/browse/YARN-5516
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Sangeetha Abdu Jyothi
>Assignee: Sean Po
> Attachments: YARN-5516.v001.patch, YARN-5516.v002.patch, 
> YARN-5516.v003.patch
>
>
> YARN-5516 changing REST API of the reservation system to support periodicity. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-7311) TestRMWebServicesReservation doesn't really test fair scheduler

2017-10-17 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-7311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16208154#comment-16208154
 ] 

Sean Po commented on YARN-7311:
---

Thanks [~yufeigu] for working on this patch. The changes in the tests look good 
to me, +1.

> TestRMWebServicesReservation doesn't really test fair scheduler
> ---
>
> Key: YARN-7311
> URL: https://issues.apache.org/jira/browse/YARN-7311
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: fairscheduler, reservation system
>Reporter: Yufei Gu
>Assignee: Yufei Gu
> Attachments: YARN-7311.001.patch, YARN-7311.002.patch, 
> YARN-7311.WIP.patch
>
>
> YARN-4248 introduced the REST API for submit/update/delete Reservations. 
> Class {{TestRMWebServicesReservation}} intends to test both FS and CS. The 
> test cases designed for fair scheduler actually test capacity scheduler. The 
> following code in method {{configureServlets}} shows it sets the scheduler to 
> CS even test cases are for fair scheduler.
> {code}
>   conf.setClass(YarnConfiguration.RM_SCHEDULER, CapacityScheduler.class,
>   ResourceScheduler.class);
>   CapacitySchedulerConfiguration csconf =
>   new CapacitySchedulerConfiguration(conf);
>   String[] queues = { "default", "dedicated" };
>   csconf.setQueues("root", queues);
>   csconf.setCapacity("root.default", 50.0f);
>   csconf.setCapacity("root.dedicated", 50.0f);
>   csconf.setReservable("root.dedicated", true);
> {code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Comment Edited] (YARN-5516) Add REST API for periodicity

2017-10-16 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5516?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16205492#comment-16205492
 ] 

Sean Po edited comment on YARN-5516 at 10/16/17 7:01 AM:
-

Verified that the tests that failed in the last run were unrelated to the 
patch. Added test to ensure the recurrence expression makes it all the way to 
the plan from REST API.


was (Author: seanpo03):
Verified that the tests that failed in the last run were unrelated to the 
patch. Added test to ensure the recurrence expression makes it all the way to 
the plan.

> Add REST API for periodicity
> 
>
> Key: YARN-5516
> URL: https://issues.apache.org/jira/browse/YARN-5516
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Sangeetha Abdu Jyothi
>Assignee: Sean Po
> Attachments: YARN-5516.v001.patch, YARN-5516.v002.patch
>
>
> YARN-5516 changing REST API of the reservation system to support periodicity. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5516) Add REST API for periodicity

2017-10-16 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5516?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5516:
--
Attachment: YARN-5516.v002.patch

Verified that the tests that failed in the last run were unrelated to the 
patch. Added test to ensure the recurrence expression makes it all the way to 
the plan.

> Add REST API for periodicity
> 
>
> Key: YARN-5516
> URL: https://issues.apache.org/jira/browse/YARN-5516
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Sangeetha Abdu Jyothi
>Assignee: Sean Po
> Attachments: YARN-5516.v001.patch, YARN-5516.v002.patch
>
>
> YARN-5516 changing REST API of the reservation system to support periodicity. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5516) Add REST API for periodicity

2017-10-14 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5516?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5516:
--
Attachment: YARN-5516.v001.patch

Added first version of the patch. 

> Add REST API for periodicity
> 
>
> Key: YARN-5516
> URL: https://issues.apache.org/jira/browse/YARN-5516
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Sangeetha Abdu Jyothi
>Assignee: Sean Po
> Attachments: YARN-5516.v001.patch
>
>
> YARN-5516 changing REST API of the reservation system to support periodicity. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5329) ReservationAgent enhancements required to support recurring reservations in the YARN ReservationSystem

2017-09-26 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16181385#comment-16181385
 ] 

Sean Po commented on YARN-5329:
---

[~subru], [~curino] the test failure occurs because of the change in 
PlanningAlgorithm where _mapAllocations.put_ is swapped with 
_allocation.addInterval_. I have applied your patch, and reverted that part of 
the code, and the test cases that failed in TestRMWebServicesResevation passed.

{code:title=PlanningAlgorithm.java#allocationsToPaddedMap|borderStyle=solid}
// If period > 0...
// ...
} else {
  // Pad at the beginning
  long earliestStart = findEarliestTime(allocation.toIntervalMap());
  if (jobArrival < earliestStart) {
allocation.addInterval(new ReservationInterval(jobArrival, 
earliestStart), zeroResource);
  }

  // Pad at the beginning
  long latestEnd = findLatestTime(allocation.toIntervalMap());
  if (latestEnd < jobDeadline) {
allocation.addInterval(new ReservationInterval(latestEnd, jobDeadline),
zeroResource);
  }
}
{code}


> ReservationAgent enhancements required to support recurring reservations in 
> the YARN ReservationSystem
> --
>
> Key: YARN-5329
> URL: https://issues.apache.org/jira/browse/YARN-5329
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Subru Krishnan
>Assignee: Carlo Curino
> Attachments: YARN-5329.v0.patch, YARN-5329.v1.patch, 
> YARN-5329.v2.patch, YARN-5329.v3.patch
>
>
> YARN-5326 proposes adding native support for recurring reservations in the 
> YARN ReservationSystem. This JIRA is a sub-task to track the changes required 
> in ReservationAgent to accomplish it. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-6528) Add JMX metrics for Plan Follower and Agent Placement and Plan Operations

2017-06-08 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-6528?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16043202#comment-16043202
 ] 

Sean Po commented on YARN-6528:
---

There are three check style failures:

The first two are unavoidable without making a dramatic API change to 
InMemoryPlan.

The last one is about about setting ReservationQueueMetrics to be a final 
class. This cannot be avoided because final classes cannot be mocked, and the 
following tests will fail:

# 
hadoop.yarn.server.resourcemanager.reservation.planning.TestSimpleCapacityReplanner
# hadoop.yarn.server.resourcemanager.reservation.TestInMemoryPlan
# 
hadoop.yarn.server.resourcemanager.reservation.planning.TestGreedyReservationAgent
# hadoop.yarn.server.resourcemanager.TestRMRestart
# hadoop.yarn.server.resourcemanager.reservation.TestCapacityOverTimePolicy
# hadoop.yarn.server.resourcemanager.reservation.planning.TestAlignedPlanner
# hadoop.yarn.server.resourcemanager.reservation.TestNoOverCommitPolicy


> Add JMX metrics for Plan Follower and Agent Placement and Plan Operations
> -
>
> Key: YARN-6528
> URL: https://issues.apache.org/jira/browse/YARN-6528
> Project: Hadoop YARN
>  Issue Type: Task
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-6528.v001.patch, YARN-6528.v002.patch, 
> YARN-6528.v003.patch, YARN-6528.v004.patch, YARN-6528.v005.patch, 
> YARN-6528.v006.patch, YARN-6528.v007.patch
>
>
> YARN-1051 introduced a ReservationSytem that enables the YARN RM to handle 
> time explicitly, i.e. users can now "reserve" capacity ahead of time which is 
> predictably allocated to them. In order to understand in finer detail the 
> performance of Rayon, YARN-6528 proposes to include JMX metrics in the Plan 
> Follower, Agent Placement and Plan Operations components of Rayon.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-6528) Add JMX metrics for Plan Follower and Agent Placement and Plan Operations

2017-06-07 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-6528?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-6528:
--
Attachment: YARN-6528.v007.patch

YARN-6528.v007.patch addresses the checkstyle issues that could be fixed. There 
are a couple that had to be left unfixed purposefully. This new patch also 
prevents double counting when an agent that internally invokes another agent is 
used.

> Add JMX metrics for Plan Follower and Agent Placement and Plan Operations
> -
>
> Key: YARN-6528
> URL: https://issues.apache.org/jira/browse/YARN-6528
> Project: Hadoop YARN
>  Issue Type: Task
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-6528.v001.patch, YARN-6528.v002.patch, 
> YARN-6528.v003.patch, YARN-6528.v004.patch, YARN-6528.v005.patch, 
> YARN-6528.v006.patch, YARN-6528.v007.patch
>
>
> YARN-1051 introduced a ReservationSytem that enables the YARN RM to handle 
> time explicitly, i.e. users can now "reserve" capacity ahead of time which is 
> predictably allocated to them. In order to understand in finer detail the 
> performance of Rayon, YARN-6528 proposes to include JMX metrics in the Plan 
> Follower, Agent Placement and Plan Operations components of Rayon.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-6528) Add JMX metrics for Plan Follower and Agent Placement and Plan Operations

2017-05-29 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-6528?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16028717#comment-16028717
 ] 

Sean Po commented on YARN-6528:
---

The test failures are currently known flaky tests tracked below:

YARN-5803 (testRMAppAttemptFailuresValidityInterval)
YARN-5548 (testRMAppAttemptFailuresValidityInterval)

The next patch will address the checkstyle issues.

> Add JMX metrics for Plan Follower and Agent Placement and Plan Operations
> -
>
> Key: YARN-6528
> URL: https://issues.apache.org/jira/browse/YARN-6528
> Project: Hadoop YARN
>  Issue Type: Task
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-6528.v001.patch, YARN-6528.v002.patch, 
> YARN-6528.v003.patch, YARN-6528.v004.patch, YARN-6528.v005.patch, 
> YARN-6528.v006.patch
>
>
> YARN-1051 introduced a ReservationSytem that enables the YARN RM to handle 
> time explicitly, i.e. users can now "reserve" capacity ahead of time which is 
> predictably allocated to them. In order to understand in finer detail the 
> performance of Rayon, YARN-6528 proposes to include JMX metrics in the Plan 
> Follower, Agent Placement and Plan Operations components of Rayon.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-6528) Add JMX metrics for Plan Follower and Agent Placement and Plan Operations

2017-05-29 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-6528?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-6528:
--
Attachment: YARN-6528.v006.patch

YARN-6528.v006.patch fixes the remaining test failures.

> Add JMX metrics for Plan Follower and Agent Placement and Plan Operations
> -
>
> Key: YARN-6528
> URL: https://issues.apache.org/jira/browse/YARN-6528
> Project: Hadoop YARN
>  Issue Type: Task
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-6528.v001.patch, YARN-6528.v002.patch, 
> YARN-6528.v003.patch, YARN-6528.v004.patch, YARN-6528.v005.patch, 
> YARN-6528.v006.patch
>
>
> YARN-1051 introduced a ReservationSytem that enables the YARN RM to handle 
> time explicitly, i.e. users can now "reserve" capacity ahead of time which is 
> predictably allocated to them. In order to understand in finer detail the 
> performance of Rayon, YARN-6528 proposes to include JMX metrics in the Plan 
> Follower, Agent Placement and Plan Operations components of Rayon.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-6528) Add JMX metrics for Plan Follower and Agent Placement and Plan Operations

2017-05-25 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-6528?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-6528:
--
Attachment: YARN-6528.v005.patch

YARN-6528.v005.patch fixes the findbugs error and the test failures. 

This patch has been tested on a single node cluster.

> Add JMX metrics for Plan Follower and Agent Placement and Plan Operations
> -
>
> Key: YARN-6528
> URL: https://issues.apache.org/jira/browse/YARN-6528
> Project: Hadoop YARN
>  Issue Type: Task
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-6528.v001.patch, YARN-6528.v002.patch, 
> YARN-6528.v003.patch, YARN-6528.v004.patch, YARN-6528.v005.patch
>
>
> YARN-1051 introduced a ReservationSytem that enables the YARN RM to handle 
> time explicitly, i.e. users can now "reserve" capacity ahead of time which is 
> predictably allocated to them. In order to understand in finer detail the 
> performance of Rayon, YARN-6528 proposes to include JMX metrics in the Plan 
> Follower, Agent Placement and Plan Operations components of Rayon.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-6528) Add JMX metrics for Plan Follower and Agent Placement and Plan Operations

2017-05-24 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-6528?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-6528:
--
Attachment: YARN-6528.v004.patch

Thanks for the comments, [~curino]. I believe the YARN-6528.v004.patch is more 
in line with what you are looking for.

> Add JMX metrics for Plan Follower and Agent Placement and Plan Operations
> -
>
> Key: YARN-6528
> URL: https://issues.apache.org/jira/browse/YARN-6528
> Project: Hadoop YARN
>  Issue Type: Task
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-6528.v001.patch, YARN-6528.v002.patch, 
> YARN-6528.v003.patch, YARN-6528.v004.patch
>
>
> YARN-1051 introduced a ReservationSytem that enables the YARN RM to handle 
> time explicitly, i.e. users can now "reserve" capacity ahead of time which is 
> predictably allocated to them. In order to understand in finer detail the 
> performance of Rayon, YARN-6528 proposes to include JMX metrics in the Plan 
> Follower, Agent Placement and Plan Operations components of Rayon.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-6528) Add JMX metrics for Plan Follower and Agent Placement and Plan Operations

2017-05-12 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-6528?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-6528:
--
Attachment: YARN-6528.v003.patch

YARN-6528.v003.patch fixes the issues that were brought up in the previous 
comment.

> Add JMX metrics for Plan Follower and Agent Placement and Plan Operations
> -
>
> Key: YARN-6528
> URL: https://issues.apache.org/jira/browse/YARN-6528
> Project: Hadoop YARN
>  Issue Type: Task
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-6528.v001.patch, YARN-6528.v002.patch, 
> YARN-6528.v003.patch
>
>
> YARN-1051 introduced a ReservationSytem that enables the YARN RM to handle 
> time explicitly, i.e. users can now "reserve" capacity ahead of time which is 
> predictably allocated to them. In order to understand in finer detail the 
> performance of Rayon, YARN-6528 proposes to include JMX metrics in the Plan 
> Follower, Agent Placement and Plan Operations components of Rayon.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-6528) Add JMX metrics for Plan Follower and Agent Placement and Plan Operations

2017-05-12 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-6528?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16008682#comment-16008682
 ] 

Sean Po commented on YARN-6528:
---

Thanks [~curino] for the comments. I'll fix 1 and 2 in the next iteration.

3. I think both can exist. Monitoring a single metric for abrupt changes to 
determine the system health is useful for automated alerts and watchdogs. 
Having metrics be posted on a per-plan basis is useful for users of the 
particular plan, as well as for administrators of the plan to determine if the 
system is in bad shape. What are your thoughts on this topic?

4. The rationale for keeping both the agent and plan metrics is to make it 
easier to determine the component responsible for a possible agent operation 
performance degradation. In addition, future agents may contain heavier 
placement logic, and it will be useful to have benchmark metrics to compare the 
agent metrics to.

> Add JMX metrics for Plan Follower and Agent Placement and Plan Operations
> -
>
> Key: YARN-6528
> URL: https://issues.apache.org/jira/browse/YARN-6528
> Project: Hadoop YARN
>  Issue Type: Task
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-6528.v001.patch, YARN-6528.v002.patch
>
>
> YARN-1051 introduced a ReservationSytem that enables the YARN RM to handle 
> time explicitly, i.e. users can now "reserve" capacity ahead of time which is 
> predictably allocated to them. In order to understand in finer detail the 
> performance of Rayon, YARN-6528 proposes to include JMX metrics in the Plan 
> Follower, Agent Placement and Plan Operations components of Rayon.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-6528) Add JMX metrics for Plan Follower and Agent Placement and Plan Operations

2017-05-08 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-6528?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-6528:
--
Attachment: YARN-6528.v002.patch

YARN-6528.v002.patch fixes the findbugs and checkstyle issues that were present 
in the previous patch. 

The reason why these patches do not include unit tests is because no new logic 
is added with the addition of this patch. Instead, components are just being 
integrated with a set of metrics.

> Add JMX metrics for Plan Follower and Agent Placement and Plan Operations
> -
>
> Key: YARN-6528
> URL: https://issues.apache.org/jira/browse/YARN-6528
> Project: Hadoop YARN
>  Issue Type: Task
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-6528.v001.patch, YARN-6528.v002.patch
>
>
> YARN-1051 introduced a ReservationSytem that enables the YARN RM to handle 
> time explicitly, i.e. users can now "reserve" capacity ahead of time which is 
> predictably allocated to them. In order to understand in finer detail the 
> performance of Rayon, YARN-6528 proposes to include JMX metrics in the Plan 
> Follower, Agent Placement and Plan Operations components of Rayon.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Comment Edited] (YARN-6528) Add JMX metrics for Plan Follower and Agent Placement and Plan Operations

2017-05-08 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-6528?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16001610#comment-16001610
 ] 

Sean Po edited comment on YARN-6528 at 5/8/17 9:48 PM:
---

YARN-6528.v002.patch fixes the findbugs and checkstyle issues that were present 
in the previous patch. 

The reason why these patches do not include unit tests is because no new logic 
is added with the addition of this patch. Instead, components are just being 
integrated with a set of metrics.

To test this patch, a single node cluster was started, and reservations were 
submitted. Each of the added metrics were verified manually through the /jmx 
page.


was (Author: seanpo03):
YARN-6528.v002.patch fixes the findbugs and checkstyle issues that were present 
in the previous patch. 

The reason why these patches do not include unit tests is because no new logic 
is added with the addition of this patch. Instead, components are just being 
integrated with a set of metrics.

> Add JMX metrics for Plan Follower and Agent Placement and Plan Operations
> -
>
> Key: YARN-6528
> URL: https://issues.apache.org/jira/browse/YARN-6528
> Project: Hadoop YARN
>  Issue Type: Task
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-6528.v001.patch, YARN-6528.v002.patch
>
>
> YARN-1051 introduced a ReservationSytem that enables the YARN RM to handle 
> time explicitly, i.e. users can now "reserve" capacity ahead of time which is 
> predictably allocated to them. In order to understand in finer detail the 
> performance of Rayon, YARN-6528 proposes to include JMX metrics in the Plan 
> Follower, Agent Placement and Plan Operations components of Rayon.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Comment Edited] (YARN-6528) Add JMX metrics for Plan Follower and Agent Placement and Plan Operations

2017-05-08 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-6528?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16001302#comment-16001302
 ] 

Sean Po edited comment on YARN-6528 at 5/8/17 6:58 PM:
---

YARN-6528.v001.patch adds metric details to the ReservationSystem, and 
abstracts common functionality between AlignedPlannerWithGreedy and 
GreedyReservationAgent. Other implementations of ReservationAgent are not 
included in the patch because the other implementations are used within the 
AlignedPlannerWithGreedy and GreedyReservationAgent. This patch has been tested 
in a single node cluster.


was (Author: seanpo03):
YARN-6528.v001.patch adds metric details to the ReservationSystem, and 
abstracts common functionality between AlignedPlannerWithGreedy and 
GreedyReservationAgent. Other implementations of ReservationAgent are not 
included in the patch because the other implementations are used within the 
AlignedPlannerWithGreedy and GreedyReservationAgent.

> Add JMX metrics for Plan Follower and Agent Placement and Plan Operations
> -
>
> Key: YARN-6528
> URL: https://issues.apache.org/jira/browse/YARN-6528
> Project: Hadoop YARN
>  Issue Type: Task
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-6528.v001.patch
>
>
> YARN-1051 introduced a ReservationSytem that enables the YARN RM to handle 
> time explicitly, i.e. users can now "reserve" capacity ahead of time which is 
> predictably allocated to them. In order to understand in finer detail the 
> performance of Rayon, YARN-6528 proposes to include JMX metrics in the Plan 
> Follower, Agent Placement and Plan Operations components of Rayon.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-6528) Add JMX metrics for Plan Follower and Agent Placement and Plan Operations

2017-05-08 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-6528?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-6528:
--
Attachment: YARN-6528.v001.patch

YARN-6528.v001.patch adds metric details to the ReservationSystem, and 
abstracts common functionality between AlignedPlannerWithGreedy and 
GreedyReservationAgent. Other implementations of ReservationAgent are not 
included in the patch because the other implementations are used within the 
AlignedPlannerWithGreedy and GreedyReservationAgent.

> Add JMX metrics for Plan Follower and Agent Placement and Plan Operations
> -
>
> Key: YARN-6528
> URL: https://issues.apache.org/jira/browse/YARN-6528
> Project: Hadoop YARN
>  Issue Type: Task
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-6528.v001.patch
>
>
> YARN-1051 introduced a ReservationSytem that enables the YARN RM to handle 
> time explicitly, i.e. users can now "reserve" capacity ahead of time which is 
> predictably allocated to them. In order to understand in finer detail the 
> performance of Rayon, YARN-6528 proposes to include JMX metrics in the Plan 
> Follower, Agent Placement and Plan Operations components of Rayon.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Assigned] (YARN-6530) Add JMX metrics for Priority Reservation Agent

2017-04-25 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-6530?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po reassigned YARN-6530:
-

Assignee: Sean Po

> Add JMX metrics for Priority Reservation Agent
> --
>
> Key: YARN-6530
> URL: https://issues.apache.org/jira/browse/YARN-6530
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacityscheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
> needed to gauge the performance of the priority reservation agent.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Resolved] (YARN-6529) Add JMX metrics for Priority Reservation Agent

2017-04-25 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-6529?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po resolved YARN-6529.
---
Resolution: Duplicate

Duplicate of YARN-6530.

> Add JMX metrics for Priority Reservation Agent
> --
>
> Key: YARN-6529
> URL: https://issues.apache.org/jira/browse/YARN-6529
> Project: Hadoop YARN
>  Issue Type: Task
>Reporter: Sean Po
>Assignee: Sean Po
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
> needed to gauge the performance of the priority reservation agent.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Created] (YARN-6530) Add JMX metrics for Priority Reservation Agent

2017-04-25 Thread Sean Po (JIRA)
Sean Po created YARN-6530:
-

 Summary: Add JMX metrics for Priority Reservation Agent
 Key: YARN-6530
 URL: https://issues.apache.org/jira/browse/YARN-6530
 Project: Hadoop YARN
  Issue Type: Sub-task
Reporter: Sean Po


YARN-5211 proposes adding support for generalized priorities for reservations 
in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
needed to gauge the performance of the priority reservation agent.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-6529) Add JMX metrics for Priority Reservation Agent

2017-04-25 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-6529?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-6529:
--
Description: YARN-5211 proposes adding support for generalized priorities 
for reservations in the YARN ReservationSystem. This JIRA is a sub-task to 
track the changes needed to gauge the performance of the priority reservation 
agent.

> Add JMX metrics for Priority Reservation Agent
> --
>
> Key: YARN-6529
> URL: https://issues.apache.org/jira/browse/YARN-6529
> Project: Hadoop YARN
>  Issue Type: Task
>Reporter: Sean Po
>Assignee: Sean Po
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
> needed to gauge the performance of the priority reservation agent.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Created] (YARN-6529) Add JMX metrics for Priority Reservation Agent

2017-04-25 Thread Sean Po (JIRA)
Sean Po created YARN-6529:
-

 Summary: Add JMX metrics for Priority Reservation Agent
 Key: YARN-6529
 URL: https://issues.apache.org/jira/browse/YARN-6529
 Project: Hadoop YARN
  Issue Type: Task
Reporter: Sean Po
Assignee: Sean Po






--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-6528) Add JMX metrics for Plan Follower and Agent Placement and Plan Operations

2017-04-25 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-6528?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-6528:
--
Description: YARN-1051 introduced a ReservationSytem that enables the YARN 
RM to handle time explicitly, i.e. users can now "reserve" capacity ahead of 
time which is predictably allocated to them. In order to understand in finer 
detail the performance of Rayon, YARN-6528 proposes to include JMX metrics in 
the Plan Follower, Agent Placement and Plan Operations components of Rayon.

> Add JMX metrics for Plan Follower and Agent Placement and Plan Operations
> -
>
> Key: YARN-6528
> URL: https://issues.apache.org/jira/browse/YARN-6528
> Project: Hadoop YARN
>  Issue Type: Task
>Reporter: Sean Po
>Assignee: Sean Po
>
> YARN-1051 introduced a ReservationSytem that enables the YARN RM to handle 
> time explicitly, i.e. users can now "reserve" capacity ahead of time which is 
> predictably allocated to them. In order to understand in finer detail the 
> performance of Rayon, YARN-6528 proposes to include JMX metrics in the Plan 
> Follower, Agent Placement and Plan Operations components of Rayon.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Created] (YARN-6528) Add JMX metrics for Plan Follower and Agent Placement and Plan Operations

2017-04-25 Thread Sean Po (JIRA)
Sean Po created YARN-6528:
-

 Summary: Add JMX metrics for Plan Follower and Agent Placement and 
Plan Operations
 Key: YARN-6528
 URL: https://issues.apache.org/jira/browse/YARN-6528
 Project: Hadoop YARN
  Issue Type: Task
Reporter: Sean Po
Assignee: Sean Po






--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Comment Edited] (YARN-5331) Extend RLESparseResourceAllocation with period for supporting recurring reservations in YARN ReservationSystem

2017-04-14 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5331?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15968207#comment-15968207
 ] 

Sean Po edited comment on YARN-5331 at 4/14/17 6:08 PM:


Thanks for the patch [~ajsangeetha], it looks good to me. +1

YARN-5816 tracks 
TestDelegationTokenRenewer.testCancelWithMultipleAppSubmissions failure
YARN-5548 tracks TestRMRestart.testFinishedAppRemovalAfterRMRestart failure


was (Author: seanpo03):
Thanks for the patch [~ajsangeetha], it looks good to me. +1

> Extend RLESparseResourceAllocation with period for supporting recurring 
> reservations in YARN ReservationSystem
> --
>
> Key: YARN-5331
> URL: https://issues.apache.org/jira/browse/YARN-5331
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Subru Krishnan
>Assignee: Sangeetha Abdu Jyothi
>  Labels: oct16-medium
> Attachments: YARN-5331.001.patch, YARN-5331.002.patch, 
> YARN-5331.003.patch, YARN-5331.004.patch, YARN-5331.005.patch, 
> YARN-5331.006.patch, YARN-5331.007.patch, YARN-5331.008.patch
>
>
> YARN-5326 proposes adding native support for recurring reservations in the 
> YARN ReservationSystem. This JIRA is a sub-task to add a 
> PeriodicRLESparseResourceAllocation. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5331) Extend RLESparseResourceAllocation with period for supporting recurring reservations in YARN ReservationSystem

2017-04-13 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5331?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15968207#comment-15968207
 ] 

Sean Po commented on YARN-5331:
---

Thanks for the patch [~ajsangeetha], it looks good to me. +1

> Extend RLESparseResourceAllocation with period for supporting recurring 
> reservations in YARN ReservationSystem
> --
>
> Key: YARN-5331
> URL: https://issues.apache.org/jira/browse/YARN-5331
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Subru Krishnan
>Assignee: Sangeetha Abdu Jyothi
>  Labels: oct16-medium
> Attachments: YARN-5331.001.patch, YARN-5331.002.patch, 
> YARN-5331.003.patch, YARN-5331.004.patch, YARN-5331.005.patch, 
> YARN-5331.006.patch, YARN-5331.007.patch, YARN-5331.008.patch
>
>
> YARN-5326 proposes adding native support for recurring reservations in the 
> YARN ReservationSystem. This JIRA is a sub-task to add a 
> PeriodicRLESparseResourceAllocation. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5331) Extend RLESparseResourceAllocation with period for supporting recurring reservations in YARN ReservationSystem

2017-03-23 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5331?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15939094#comment-15939094
 ] 

Sean Po commented on YARN-5331:
---

Thanks [~ajsangeetha] for the update, a few more comments:
* It may be more appropriate to log instead of printing to stdout using 
System.out.println. (PeriodicRLESparseResourceAllocation.java+84,99)
* NIT: PeriodicRLESparseResourceAllocation::removeInterval does not have the 
same checks for time interval that 
PeriodicRLESparseResourceAllocation::setCapacityInInterval does.
* We have checks in PeriodicRLESparseResourceAllocation::setCapacityInInterval 
and PeriodicRLESparseResourceAllocation::removeInterval that will prevent the 
capacity from being added or removed. Can we verify these checks by adding test 
cases that would fail to be added or removed?
* Would it be better to have 
PeriodicRLESparseResourceAllocation::setCapacityInInterval or 
PeriodicRLESparseResourceAllocation::removeInterval return bool, to allow the 
caller to determine whether or not the method call succeeded?
* NIT: Please add a test case in 
PeriodicRLESparseResourceAllocation::testPeriodicCapacity, 
PeriodicRLESparseResourceAllocation::testRemoveInterval that does not land on a 
timeStep.
* Do we need to handle the possibility of negative periodic capacity when 
PeriodicRLESparseResourceAllocation::removeInterval is used?


> Extend RLESparseResourceAllocation with period for supporting recurring 
> reservations in YARN ReservationSystem
> --
>
> Key: YARN-5331
> URL: https://issues.apache.org/jira/browse/YARN-5331
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Subru Krishnan
>Assignee: Sangeetha Abdu Jyothi
>  Labels: oct16-medium
> Attachments: YARN-5331.001.patch, YARN-5331.002.patch, 
> YARN-5331.003.patch, YARN-5331.004.patch, YARN-5331.005.patch
>
>
> YARN-5326 proposes adding native support for recurring reservations in the 
> YARN ReservationSystem. This JIRA is a sub-task to add a 
> PeriodicRLESparseResourceAllocation. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5703) ReservationAgents are not correctly configured

2017-03-06 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5703?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15898630#comment-15898630
 ] 

Sean Po commented on YARN-5703:
---

I apologize for not responding,  [~maniraj...@gmail.com] and [~Naganarasimha]. 
I haven't been checking my email for the last few weeks for personal reasons. 
Thank you both for contributing, and checking this in!

> ReservationAgents are not correctly configured
> --
>
> Key: YARN-5703
> URL: https://issues.apache.org/jira/browse/YARN-5703
> Project: Hadoop YARN
>  Issue Type: Bug
>  Components: capacity scheduler, resourcemanager
>Affects Versions: 3.0.0-alpha1
>Reporter: Sean Po
>Assignee: Manikandan R
> Attachments: YARN-5703.001.patch, YARN-5703.002.patch, 
> YARN-5703.003.patch, YARN-5703.004.patch, YARN-5703.005.patch, 
> YARN-5703.006.patch, YARN-5703.007.patch, YARN-5703.008.patch
>
>
> In AbstractReservationSystem, the method that instantiates a ReservationAgent 
> does not properly initialize it with the appropriate configuration because it 
> expects the ReservationAgent to implement Configurable.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Assigned] (YARN-5881) Enable configuration of queue capacity in terms of absolute resources

2017-03-06 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5881?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po reassigned YARN-5881:
-

Assignee: Wangda Tan  (was: Sean Po)

> Enable configuration of queue capacity in terms of absolute resources
> -
>
> Key: YARN-5881
> URL: https://issues.apache.org/jira/browse/YARN-5881
> Project: Hadoop YARN
>  Issue Type: Improvement
>Reporter: Sean Po
>Assignee: Wangda Tan
> Attachments: 
> YARN-5881.Support.Absolute.Min.Max.Resource.In.Capacity.Scheduler.design-doc.v1.pdf
>
>
> Currently, Yarn RM supports the configuration of queue capacity in terms of a 
> proportion to cluster capacity. In the context of Yarn being used as a public 
> cloud service, it makes more sense if queues can be configured absolutely. 
> This will allow administrators to set usage limits more concretely and 
> simplify customer expectations for cluster allocation.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5881) Enable configuration of queue capacity in terms of absolute resources

2017-03-06 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5881?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15898625#comment-15898625
 ] 

Sean Po commented on YARN-5881:
---

Thanks Wangda, please take over the ticket. I have assigned it to you.

> Enable configuration of queue capacity in terms of absolute resources
> -
>
> Key: YARN-5881
> URL: https://issues.apache.org/jira/browse/YARN-5881
> Project: Hadoop YARN
>  Issue Type: Improvement
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: 
> YARN-5881.Support.Absolute.Min.Max.Resource.In.Capacity.Scheduler.design-doc.v1.pdf
>
>
> Currently, Yarn RM supports the configuration of queue capacity in terms of a 
> proportion to cluster capacity. In the context of Yarn being used as a public 
> cloud service, it makes more sense if queues can be configured absolutely. 
> This will allow administrators to set usage limits more concretely and 
> simplify customer expectations for cluster allocation.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5385) Add a PriorityAgent in ReservationSystem

2017-02-27 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5385?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15887042#comment-15887042
 ] 

Sean Po commented on YARN-5385:
---

Two of the test failures are accounted for with a JIRA.

YARN-6231 - 
hadoop.yarn.server.resourcemanager.scheduler.fair.TestFairScheduler::testMoveWouldViolateMaxResourcesConstraints
YARN-5548 - 
org.apache.hadoop.yarn.server.resourcemanager.TestRMRestart::testFinishedAppRemovalAfterRMRestart

The remaining failure 
hadoop.yarn.server.resourcemanager.scheduler.fair.TestFairSchedulerPreemption::testPreemptionSelectNonAMContainer
 appears to be a flaky test. A local run of the test was very inconsistent, 
with some fails and some successes across multiple tests. YARN-6249 was created 
to track this problem. 

> Add a PriorityAgent in ReservationSystem 
> -
>
> Key: YARN-5385
> URL: https://issues.apache.org/jira/browse/YARN-5385
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
>  Labels: oct16-hard
> Attachments: YARN-5385.v002.patch, YARN-5385.v003.patch, 
> YARN-5385.v004.patch, YARN-5385.v005.patch, YARN-5385.v006.patch, 
> YARN-5385.v007.patch, YARN-5385.v1.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the addition 
> of a priority agent to accomplish it. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-6249) TestFairSchedulerPreemption is inconsistently failing on trunk

2017-02-27 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-6249?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-6249:
--
Affects Version/s: 2.9.0

> TestFairSchedulerPreemption is inconsistently failing on trunk
> --
>
> Key: YARN-6249
> URL: https://issues.apache.org/jira/browse/YARN-6249
> Project: Hadoop YARN
>  Issue Type: Bug
>  Components: fairscheduler, resourcemanager
>Affects Versions: 2.9.0
>Reporter: Sean Po
>
> Tests in TestFairSchedulerPreemption.java will inconsistently fail on trunk. 
> An example stack trace: 
> Tests run: 24, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 24.879 sec 
> <<< FAILURE! - in 
> org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.TestFairSchedulerPreemption
> testPreemptionSelectNonAMContainer[MinSharePreemptionWithDRF](org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.TestFairSchedulerPreemption)
>   Time elapsed: 10.475 sec  <<< FAILURE!
> java.lang.AssertionError: Incorrect number of containers on the greedy app 
> expected:<4> but was:<8>
>   at org.junit.Assert.fail(Assert.java:88)
>   at org.junit.Assert.failNotEquals(Assert.java:743)
>   at org.junit.Assert.assertEquals(Assert.java:118)
>   at org.junit.Assert.assertEquals(Assert.java:555)
>   at 
> org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.TestFairSchedulerPreemption.verifyPreemption(TestFairSchedulerPreemption.java:288)
>   at 
> org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.TestFairSchedulerPreemption.testPreemptionSelectNonAMContainer(TestFairSchedulerPreemption.java:363)



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Created] (YARN-6249) TestFairSchedulerPreemption is inconsistently failing on trunk

2017-02-27 Thread Sean Po (JIRA)
Sean Po created YARN-6249:
-

 Summary: TestFairSchedulerPreemption is inconsistently failing on 
trunk
 Key: YARN-6249
 URL: https://issues.apache.org/jira/browse/YARN-6249
 Project: Hadoop YARN
  Issue Type: Bug
  Components: fairscheduler, resourcemanager
Reporter: Sean Po


Tests in TestFairSchedulerPreemption.java will inconsistently fail on trunk. An 
example stack trace: 

Tests run: 24, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 24.879 sec <<< 
FAILURE! - in 
org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.TestFairSchedulerPreemption
testPreemptionSelectNonAMContainer[MinSharePreemptionWithDRF](org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.TestFairSchedulerPreemption)
  Time elapsed: 10.475 sec  <<< FAILURE!
java.lang.AssertionError: Incorrect number of containers on the greedy app 
expected:<4> but was:<8>
at org.junit.Assert.fail(Assert.java:88)
at org.junit.Assert.failNotEquals(Assert.java:743)
at org.junit.Assert.assertEquals(Assert.java:118)
at org.junit.Assert.assertEquals(Assert.java:555)
at 
org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.TestFairSchedulerPreemption.verifyPreemption(TestFairSchedulerPreemption.java:288)
at 
org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.TestFairSchedulerPreemption.testPreemptionSelectNonAMContainer(TestFairSchedulerPreemption.java:363)



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5385) Add a PriorityAgent in ReservationSystem

2017-02-27 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5385?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5385:
--
Attachment: YARN-5385.v007.patch

YARN-5385.v007.patch fixes a configuration bug found while running a single 
node cluster test.

The patch also attempts to fix the find bugs error from the latest QA run. 

> Add a PriorityAgent in ReservationSystem 
> -
>
> Key: YARN-5385
> URL: https://issues.apache.org/jira/browse/YARN-5385
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
>  Labels: oct16-hard
> Attachments: YARN-5385.v002.patch, YARN-5385.v003.patch, 
> YARN-5385.v004.patch, YARN-5385.v005.patch, YARN-5385.v006.patch, 
> YARN-5385.v007.patch, YARN-5385.v1.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the addition 
> of a priority agent to accomplish it. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5385) Add a PriorityAgent in ReservationSystem

2017-02-23 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5385?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5385:
--
Attachment: YARN-5385.v006.patch

YARN-5385.v006.patch attempts to fix the findbugs errors by wrapping the 
methods being locked, with a try finally block.

> Add a PriorityAgent in ReservationSystem 
> -
>
> Key: YARN-5385
> URL: https://issues.apache.org/jira/browse/YARN-5385
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
>  Labels: oct16-hard
> Attachments: YARN-5385.v002.patch, YARN-5385.v003.patch, 
> YARN-5385.v004.patch, YARN-5385.v005.patch, YARN-5385.v006.patch, 
> YARN-5385.v1.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the addition 
> of a priority agent to accomplish it. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Comment Edited] (YARN-5385) Add a PriorityAgent in ReservationSystem

2017-02-23 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5385?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15881843#comment-15881843
 ] 

Sean Po edited comment on YARN-5385 at 2/24/17 3:03 AM:


The two test failures are known test failures tracked in JIRA: 
YARN-5548 - org.apache.hadoop.yarn.server.resourcemanager.TestRMRestart
testFinishedAppRemovalAfterRMRestart

YARN-5816 - 
org.apache.hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer
testCancelWithMultipleAppSubmissions




was (Author: seanpo03):
The two test failures are: 


> Add a PriorityAgent in ReservationSystem 
> -
>
> Key: YARN-5385
> URL: https://issues.apache.org/jira/browse/YARN-5385
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
>  Labels: oct16-hard
> Attachments: YARN-5385.v002.patch, YARN-5385.v003.patch, 
> YARN-5385.v004.patch, YARN-5385.v005.patch, YARN-5385.v1.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the addition 
> of a priority agent to accomplish it. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5385) Add a PriorityAgent in ReservationSystem

2017-02-23 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5385?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15881843#comment-15881843
 ] 

Sean Po commented on YARN-5385:
---

The two test failures are: 


> Add a PriorityAgent in ReservationSystem 
> -
>
> Key: YARN-5385
> URL: https://issues.apache.org/jira/browse/YARN-5385
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
>  Labels: oct16-hard
> Attachments: YARN-5385.v002.patch, YARN-5385.v003.patch, 
> YARN-5385.v004.patch, YARN-5385.v005.patch, YARN-5385.v1.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the addition 
> of a priority agent to accomplish it. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5385) Add a PriorityAgent in ReservationSystem

2017-02-23 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5385?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5385:
--
Attachment: YARN-5385.v005.patch

Thanks [~curino] for the review! I have addressed most of the comments you have 
given me. My comments below:

1. The CapacitySchedulerConfiguration implements the get by queue method.

2. I have changed the words 'accommodate for' in code and comments to 'make 
room for'.

3. In our clusters, we currently do not see many priority inversions. For our 
use, it seems that this log won't be too verbose, and having the logs will be 
more beneficial than harmful. 

4. The makeRoomForReservation method is abstract for PriorityReservationAgent 
because subclasses will want to determine 2 things. 1, the order for deletion, 
and 2, the order for re-addition. I have documented this in the java doc to 
make it more clear that makeRoomForReservation is meant to return an ordered 
list of reservation allocations.

5. This problem will require a lot of work, and it may be more appropriate for 
this to be completed in a different task. I have created YARN-6227 to track 
this feature.

7. I have added locks around the implementation of the ReservationAgent API for 
the PriorityReservationAgent. I understand this solution may not be the best in 
the event that we get a lot of multi-threaded access in a short period of time, 
but a more appropriate solution will be complicated, and it may be better if it 
is done in a separate task. YARN-6226 was created to track this improvement.

10. In RMWebService, if the priority is not given, it will be set to an 
undefined value. PriorityReservationAgent converts undefined priority to the 
lowest possible priority.

11. I do have tests that check invariants.

> Add a PriorityAgent in ReservationSystem 
> -
>
> Key: YARN-5385
> URL: https://issues.apache.org/jira/browse/YARN-5385
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
>  Labels: oct16-hard
> Attachments: YARN-5385.v002.patch, YARN-5385.v003.patch, 
> YARN-5385.v004.patch, YARN-5385.v005.patch, YARN-5385.v1.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the addition 
> of a priority agent to accomplish it. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Created] (YARN-6227) Information about reservations that are yielded by higher priority reservations need to be surfaced to the client

2017-02-23 Thread Sean Po (JIRA)
Sean Po created YARN-6227:
-

 Summary: Information about reservations that are yielded by higher 
priority reservations need to be surfaced to the client
 Key: YARN-6227
 URL: https://issues.apache.org/jira/browse/YARN-6227
 Project: Hadoop YARN
  Issue Type: Task
Reporter: Sean Po


YARN-5385 introduces a PriorityReservationAgent that can be used to yield lower 
priority reservations so that higher priority reservations may fit if it 
initially could not be created in the plan. In the event that a low priority 
reservation is yielded, the client does not have a way to know prior to using 
the reservation that it has been removed. This JIRA aims to surface such 
information to the client, so that it may be able to act appropriately to its 
lost reservation.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Created] (YARN-6226) Improve concurrency model in PriorityReservationAgent

2017-02-23 Thread Sean Po (JIRA)
Sean Po created YARN-6226:
-

 Summary: Improve concurrency model in PriorityReservationAgent
 Key: YARN-6226
 URL: https://issues.apache.org/jira/browse/YARN-6226
 Project: Hadoop YARN
  Issue Type: Task
  Components: resourcemanager
Reporter: Sean Po
Assignee: Sean Po


YARN-5385 introduces a PriorityReservationAgent that can be used to yield lower 
priority reservations so that higher priority reservations may fit if it 
initially could not be created in the plan. In YARN-5385, a write lock was used 
in the createReservation, updateReservation and deleteReservation methods to 
ensure that multi-threaded access will not be a problem for the new agent. This 
solution may be problematic if many multi-threaded reservation operations are 
made in a short period of time due to the heaviness of the createReservation 
and updateReservation methods. This JIRA aims to improve the concurrency model 
of the PriorityReservationAgent by replacing the write lock with a more 
suitable solution.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5703) ReservationAgents are not correctly configured

2017-01-26 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5703?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15840458#comment-15840458
 ] 

Sean Po commented on YARN-5703:
---

Hi [~maniraj...@gmail.com], thanks for your interest in this. Please feel free 
to work on this JIRA.

> ReservationAgents are not correctly configured
> --
>
> Key: YARN-5703
> URL: https://issues.apache.org/jira/browse/YARN-5703
> Project: Hadoop YARN
>  Issue Type: Bug
>  Components: capacity scheduler, resourcemanager
>Affects Versions: 3.0.0-alpha1
>Reporter: Sean Po
>Assignee: Sean Po
>
> In AbstractReservationSystem, the method that instantiates a ReservationAgent 
> does not properly initialize it with the appropriate configuration because it 
> expects the ReservationAgent to implement Configurable.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5839) ClusterApplication API does not include a ReservationID

2016-11-29 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5839?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5839:
--
Attachment: YARN-5839.v003.patch

YARN-5839.v003.patch fixes unit tests, Javadoc, and checkstyle. There is one 
checkstyle that remains. It cannot be avoided without unrelated refactoring.

> ClusterApplication API does not include a ReservationID
> ---
>
> Key: YARN-5839
> URL: https://issues.apache.org/jira/browse/YARN-5839
> Project: Hadoop YARN
>  Issue Type: Improvement
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5839.v001.patch, YARN-5839.v002.patch, 
> YARN-5839.v003.patch
>
>
> Currently, the ClusterApplication, and ClusterApplications API does not allow 
> users to find the reservation queue that an application is running in. 
> YARN-5839 proposes to add ReservationId to the ClusterApplication and 
> ClusterApplications API.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5839) ClusterApplication API does not include a ReservationID

2016-11-29 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5839?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5839:
--
Attachment: YARN-5839.v002.patch

Re-create the patch from latest trunk.

> ClusterApplication API does not include a ReservationID
> ---
>
> Key: YARN-5839
> URL: https://issues.apache.org/jira/browse/YARN-5839
> Project: Hadoop YARN
>  Issue Type: Improvement
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5839.v001.patch, YARN-5839.v002.patch
>
>
> Currently, the ClusterApplication, and ClusterApplications API does not allow 
> users to find the reservation queue that an application is running in. 
> YARN-5839 proposes to add ReservationId to the ClusterApplication and 
> ClusterApplications API.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5435) [Regression] QueueCapacities not being updated for dynamic ReservationQueue

2016-11-28 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5435?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5435:
--
Attachment: YARN-5435.v005.patch

YARN-5435.v005.patch modifies the code so that all tests pass, and only max 
QueueCapacities are inherited from the parent queue.

> [Regression] QueueCapacities not being updated for dynamic ReservationQueue
> ---
>
> Key: YARN-5435
> URL: https://issues.apache.org/jira/browse/YARN-5435
> Project: Hadoop YARN
>  Issue Type: Bug
>  Components: capacity scheduler, resourcemanager
>Affects Versions: 2.8.0
>Reporter: Sean Po
>Assignee: Sean Po
>  Labels: oct16-easy, regression
> Attachments: YARN-5435.v003.patch, YARN-5435.v004.patch, 
> YARN-5435.v005.patch, YARN-5435.v1.patch, YARN-5435.v2.patch
>
>
> YARN-1707 added dynamic queues (ReservationQueue) to CapacityScheduler. The 
> QueueCapacities data structure was added subsequently but is not being 
> updated correctly for ReservationQueue. This JIRA tracks the changes required 
> to update QueueCapacities of ReservationQueue correctly.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5839) ClusterApplication API does not include a ReservationID

2016-11-28 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5839?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5839:
--
Description: Currently, the ClusterApplication, and ClusterApplications API 
does not allow users to find the reservation queue that an application is 
running in. YARN-5839 proposes to add ReservationId to the ClusterApplication 
and ClusterApplications API.  (was: Currently, the ClusterApplication, and 
ClusterApplications API does not allow users to find the reservation queue that 
an application is running in. YARN-5435 proposes to add ReservationId to the 
ClusterApplication and ClusterApplications API.)

> ClusterApplication API does not include a ReservationID
> ---
>
> Key: YARN-5839
> URL: https://issues.apache.org/jira/browse/YARN-5839
> Project: Hadoop YARN
>  Issue Type: Improvement
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5839.v001.patch
>
>
> Currently, the ClusterApplication, and ClusterApplications API does not allow 
> users to find the reservation queue that an application is running in. 
> YARN-5839 proposes to add ReservationId to the ClusterApplication and 
> ClusterApplications API.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Comment Edited] (YARN-5839) ClusterApplication API does not include a ReservationID

2016-11-28 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5839?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15703855#comment-15703855
 ] 

Sean Po edited comment on YARN-5839 at 11/29/16 1:53 AM:
-

YARN-5839.v001.patch includes the changes to the REST and RPC API to add a 
ReservationId to the ApplicationReport.


was (Author: seanpo03):
YARN-5389.v001.patch includes the changes to the REST and RPC API to add a 
ReservationId to the ApplicationReport.

> ClusterApplication API does not include a ReservationID
> ---
>
> Key: YARN-5839
> URL: https://issues.apache.org/jira/browse/YARN-5839
> Project: Hadoop YARN
>  Issue Type: Improvement
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5839.v001.patch
>
>
> Currently, the ClusterApplication, and ClusterApplications API does not allow 
> users to find the reservation queue that an application is running in. 
> YARN-5435 proposes to add ReservationId to the ClusterApplication and 
> ClusterApplications API.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5839) ClusterApplication API does not include a ReservationID

2016-11-28 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5839?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5839:
--
Attachment: YARN-5839.v001.patch

> ClusterApplication API does not include a ReservationID
> ---
>
> Key: YARN-5839
> URL: https://issues.apache.org/jira/browse/YARN-5839
> Project: Hadoop YARN
>  Issue Type: Improvement
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5839.v001.patch
>
>
> Currently, the ClusterApplication, and ClusterApplications API does not allow 
> users to find the reservation queue that an application is running in. 
> YARN-5435 proposes to add ReservationId to the ClusterApplication and 
> ClusterApplications API.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Created] (YARN-5881) Enable configuration of queue capacity in terms of absolute resources

2016-11-14 Thread Sean Po (JIRA)
Sean Po created YARN-5881:
-

 Summary: Enable configuration of queue capacity in terms of 
absolute resources
 Key: YARN-5881
 URL: https://issues.apache.org/jira/browse/YARN-5881
 Project: Hadoop YARN
  Issue Type: Improvement
Reporter: Sean Po
Assignee: Sean Po


Currently, Yarn RM supports the configuration of queue capacity in terms of a 
proportion to cluster capacity. In the context of Yarn being used as a public 
cloud service, it makes more sense if queues can be configured absolutely. This 
will allow administrators to set usage limits more concretely and simplify 
customer expectations for cluster allocation.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Created] (YARN-5839) ClusterApplication API does not include a ReservationID

2016-11-04 Thread Sean Po (JIRA)
Sean Po created YARN-5839:
-

 Summary: ClusterApplication API does not include a ReservationID
 Key: YARN-5839
 URL: https://issues.apache.org/jira/browse/YARN-5839
 Project: Hadoop YARN
  Issue Type: Improvement
Reporter: Sean Po
Assignee: Sean Po


Currently, the ClusterApplication, and ClusterApplications API does not allow 
users to find the reservation queue that an application is running in. 
YARN-5435 proposes to add ReservationId to the ClusterApplication and 
ClusterApplications API.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5435) [Regression] QueueCapacities not being updated for dynamic ReservationQueue

2016-10-28 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5435?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5435:
--
Attachment: YARN-5435.v004.patch

Thanks [~curino]. I added the change you suggested, and updated the tests to 
pass in the lastest change.

> [Regression] QueueCapacities not being updated for dynamic ReservationQueue
> ---
>
> Key: YARN-5435
> URL: https://issues.apache.org/jira/browse/YARN-5435
> Project: Hadoop YARN
>  Issue Type: Bug
>  Components: capacity scheduler, resourcemanager
>Affects Versions: 2.8.0
>Reporter: Sean Po
>Assignee: Sean Po
>  Labels: oct16-easy, regression
> Attachments: YARN-5435.v003.patch, YARN-5435.v004.patch, 
> YARN-5435.v1.patch, YARN-5435.v2.patch
>
>
> YARN-1707 added dynamic queues (ReservationQueue) to CapacityScheduler. The 
> QueueCapacities data structure was added subsequently but is not being 
> updated correctly for ReservationQueue. This JIRA tracks the changes required 
> to update QueueCapacities of ReservationQueue correctly.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5435) [Regression] QueueCapacities not being updated for dynamic ReservationQueue

2016-10-27 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5435?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5435:
--
Attachment: YARN-5435.v003.patch

Thanks [~curino] for the review. This latest patch uses the CSQueueUtils class 
to inherit properties from the parent queue. It also adds the ability to 
whitelist labels for inheritance so that we can select just NO_LABEL to 
inherit. 

> [Regression] QueueCapacities not being updated for dynamic ReservationQueue
> ---
>
> Key: YARN-5435
> URL: https://issues.apache.org/jira/browse/YARN-5435
> Project: Hadoop YARN
>  Issue Type: Bug
>  Components: capacity scheduler, resourcemanager
>Affects Versions: 2.8.0
>Reporter: Sean Po
>Assignee: Sean Po
>  Labels: oct16-easy, regression
> Attachments: YARN-5435.v003.patch, YARN-5435.v1.patch, 
> YARN-5435.v2.patch
>
>
> YARN-1707 added dynamic queues (ReservationQueue) to CapacityScheduler. The 
> QueueCapacities data structure was added subsequently but is not being 
> updated correctly for ReservationQueue. This JIRA tracks the changes required 
> to update QueueCapacities of ReservationQueue correctly.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5600) Add a parameter to ContainerLaunchContext to emulate yarn.nodemanager.delete.debug-delay-sec on a per-application basis

2016-10-27 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5600?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5600:
--
Labels: oct16-medium  (was: )

> Add a parameter to ContainerLaunchContext to emulate 
> yarn.nodemanager.delete.debug-delay-sec on a per-application basis
> ---
>
> Key: YARN-5600
> URL: https://issues.apache.org/jira/browse/YARN-5600
> Project: Hadoop YARN
>  Issue Type: Improvement
>  Components: nodemanager
>Affects Versions: 3.0.0-alpha1
>Reporter: Daniel Templeton
>Assignee: Miklos Szegedi
>  Labels: oct16-medium
> Attachments: YARN-5600.000.patch, YARN-5600.001.patch, 
> YARN-5600.002.patch
>
>
> To make debugging application launch failures simpler, I'd like to add a 
> parameter to the CLC to allow an application owner to request delayed 
> deletion of the application's launch artifacts.
> This JIRA solves largely the same problem as YARN-5599, but for cases where 
> ATS is not in use, e.g. branch-2.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5705) [YARN-3368] Add support for Timeline V2 to new web UI

2016-10-27 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5705?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5705:
--
Labels: oct16-hard  (was: )

> [YARN-3368] Add support for Timeline V2 to new web UI
> -
>
> Key: YARN-5705
> URL: https://issues.apache.org/jira/browse/YARN-5705
> Project: Hadoop YARN
>  Issue Type: Sub-task
>Reporter: Sunil G
>Assignee: Akhil PB
>  Labels: oct16-hard
> Attachments: YARN-5705.001.patch, YARN-5705.002.patch, 
> YARN-5705.003.patch, YARN-5705.004.patch, YARN-5705.005.patch, 
> YARN-5705.006.patch, YARN-5705.007.patch, YARN-5705.008.patch, 
> YARN-5705.009.patch, YARN-5705.010.patch, YARN-5705.011.patch, 
> YARN-5705.012.patch
>
>
> Integrate timeline v2 to YARN-3368. This is a clone JIRA for YARN-4097



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5722) FairScheduler hides group resolution exceptions when assigning queue

2016-10-27 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5722?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5722:
--
Labels: oct16-easy  (was: )

> FairScheduler hides group resolution exceptions when assigning queue 
> -
>
> Key: YARN-5722
> URL: https://issues.apache.org/jira/browse/YARN-5722
> Project: Hadoop YARN
>  Issue Type: Improvement
>  Components: fairscheduler
>Affects Versions: 2.6.5, 3.0.0-alpha1
>Reporter: Wilfred Spiegelenburg
>Assignee: Wilfred Spiegelenburg
>  Labels: oct16-easy
> Attachments: YARN-5722.1.patch
>
>
> When a group based placement rule is used and the user does not have any 
> groups the reason for rejecting the application is hidden. An assignment will 
> fail as follows:
> {code}
>  
>  
> {code}
> The error logged on the client side:
> {code}
> 09/30 15:59:27 INFO mapreduce.JobSubmitter: Cleaning up the staging area 
> /user/test_user/.staging/job_1475223610304_6043 
> 16/09/30 15:59:27 WARN security.UserGroupInformation: 
> PriviledgedActionException as:test_user (auth:SIMPLE) 
> cause:java.io.IOException: Failed to run job : Error assigning app to queue 
> default 
> java.io.IOException: Failed to run job : Error assigning app to queue default 
> at org.apache.hadoop.mapred.YARNRunner.submitJob(YARNRunner.java:301) 
> at 
> org.apache.hadoop.mapreduce.JobSubmitter.submitJobInternal(JobSubmitter.java:244)
>  
> at org.apache.hadoop.mapreduce.Job$10.run(Job.java:1307) 
> {code}
> The {{default}} queue name is passed in as part of the application submission 
> and not really the queue that is tried.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5767) Fix the order that resources are cleaned up from the local Public/Private caches

2016-10-27 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5767?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5767:
--
Labels: oct16-easy  (was: )

> Fix the order that resources are cleaned up from the local Public/Private 
> caches
> 
>
> Key: YARN-5767
> URL: https://issues.apache.org/jira/browse/YARN-5767
> Project: Hadoop YARN
>  Issue Type: Bug
>Affects Versions: 2.6.0, 2.7.0, 3.0.0-alpha1
>Reporter: Chris Trezzo
>Assignee: Chris Trezzo
>  Labels: oct16-easy
> Attachments: YARN-5767-trunk-v1.patch, YARN-5767-trunk-v2.patch, 
> YARN-5767-trunk-v3.patch, YARN-5767-trunk-v4.patch
>
>
> If you look at {{ResourceLocalizationService#handleCacheCleanup}}, you can 
> see that public resources are added to the {{ResourceRetentionSet}} first 
> followed by private resources:
> {code:java}
> private void handleCacheCleanup(LocalizationEvent event) {
>   ResourceRetentionSet retain =
> new ResourceRetentionSet(delService, cacheTargetSize);
>   retain.addResources(publicRsrc);
>   if (LOG.isDebugEnabled()) {
> LOG.debug("Resource cleanup (public) " + retain);
>   }
>   for (LocalResourcesTracker t : privateRsrc.values()) {
> retain.addResources(t);
> if (LOG.isDebugEnabled()) {
>   LOG.debug("Resource cleanup " + t.getUser() + ":" + retain);
> }
>   }
>   //TODO Check if appRsrcs should also be added to the retention set.
> }
> {code}
> Unfortunately, if we look at {{ResourceRetentionSet#addResources}} we see 
> that this means public resources are deleted first until the target cache 
> size is met:
> {code:java}
> public void addResources(LocalResourcesTracker newTracker) {
>   for (LocalizedResource resource : newTracker) {
> currentSize += resource.getSize();
> if (resource.getRefCount() > 0) {
>   // always retain resources in use
>   continue;
> }
> retain.put(resource, newTracker);
>   }
>   for (Iterator> i =
>  retain.entrySet().iterator();
>currentSize - delSize > targetSize && i.hasNext();) {
> Map.Entry rsrc = i.next();
> LocalizedResource resource = rsrc.getKey();
> LocalResourcesTracker tracker = rsrc.getValue();
> if (tracker.remove(resource, delService)) {
>   delSize += resource.getSize();
>   i.remove();
> }
>   }
> }
> {code}
> The result of this is that resources in the private cache are only deleted in 
> the cases where:
> # The cache size is larger than the target cache size and the public cache is 
> empty.
> # The cache size is larger than the target cache size and everything in the 
> public cache is being used by a running container.
> For clusters that primarily use the public cache (i.e. make use of the shared 
> cache), this means that the most commonly used resources can be deleted 
> before old resources in the private cache. Furthermore, the private cache can 
> continue to grow over time causing more and more churn in the public cache.
> Additionally, the same problem exists within the private cache. Since 
> resources are added to the retention set on a user by user basis, resources 
> will get cleaned up one user at a time in the order that privateRsrc.values() 
> returns the LocalResourcesTracker. So if user1 has 10MB in their cache and 
> user2 has 100MB in their cache and the target size of the cache is 50MB, 
> user1 could potentially have their entire cache removed before anything is 
> deleted from the user2 cache.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Created] (YARN-5763) HttpListener takes upwards of 3 minutes to start on failover

2016-10-20 Thread Sean Po (JIRA)
Sean Po created YARN-5763:
-

 Summary: HttpListener takes upwards of 3 minutes to start on 
failover
 Key: YARN-5763
 URL: https://issues.apache.org/jira/browse/YARN-5763
 Project: Hadoop YARN
  Issue Type: Bug
Reporter: Sean Po
Assignee: Sean Po


When Yarn RM fails over to another instance, it takes multiple minutes before 
the new master Yarn RM can begin accepting requests.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5385) Add a PriorityAgent in ReservationSystem

2016-10-10 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5385?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15563668#comment-15563668
 ] 

Sean Po commented on YARN-5385:
---

The final Javac warning about unchecked conversion in RMWebServices is 
unrelated to the changes in YARN-5385.v004.patch.

> Add a PriorityAgent in ReservationSystem 
> -
>
> Key: YARN-5385
> URL: https://issues.apache.org/jira/browse/YARN-5385
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5385.v002.patch, YARN-5385.v003.patch, 
> YARN-5385.v004.patch, YARN-5385.v1.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the addition 
> of a priority agent to accomplish it. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5385) Add a PriorityAgent in ReservationSystem

2016-10-10 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5385?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5385:
--
Attachment: YARN-5385.v004.patch

YARN-5385.v004.patch implements Serializable for the ReservationComparator.

> Add a PriorityAgent in ReservationSystem 
> -
>
> Key: YARN-5385
> URL: https://issues.apache.org/jira/browse/YARN-5385
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5385.v002.patch, YARN-5385.v003.patch, 
> YARN-5385.v004.patch, YARN-5385.v1.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the addition 
> of a priority agent to accomplish it. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5385) Add a PriorityAgent in ReservationSystem

2016-10-07 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5385?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5385:
--
Attachment: YARN-5385.v003.patch

YARN-5385.v003.patch cleans up the test for the SimplePriorityReservationAgent, 
and adds tests for different ReservationPriorityScopes.

> Add a PriorityAgent in ReservationSystem 
> -
>
> Key: YARN-5385
> URL: https://issues.apache.org/jira/browse/YARN-5385
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5385.v002.patch, YARN-5385.v003.patch, 
> YARN-5385.v1.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the addition 
> of a priority agent to accomplish it. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5385) Add a PriorityAgent in ReservationSystem

2016-10-03 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5385?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5385:
--
Attachment: YARN-5385.v002.patch

YARN-5385.v002.patch adds a priority aware reservation agent. 

I have tested this code on a single node cluster.

> Add a PriorityAgent in ReservationSystem 
> -
>
> Key: YARN-5385
> URL: https://issues.apache.org/jira/browse/YARN-5385
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5385.v002.patch, YARN-5385.v1.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the addition 
> of a priority agent to accomplish it. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Created] (YARN-5703) ReservationAgents are not correctly configured

2016-10-03 Thread Sean Po (JIRA)
Sean Po created YARN-5703:
-

 Summary: ReservationAgents are not correctly configured
 Key: YARN-5703
 URL: https://issues.apache.org/jira/browse/YARN-5703
 Project: Hadoop YARN
  Issue Type: Bug
  Components: capacity scheduler, resourcemanager
Affects Versions: 3.0.0-alpha1
Reporter: Sean Po
Assignee: Sean Po


In AbstractReservationSystem, the method that instantiates a ReservationAgent 
does not properly initialize it with the appropriate configuration because it 
expects the ReservationAgent to implement Configurable.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5384) Expose priority in ReservationSystem submission APIs

2016-09-30 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5384?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15537141#comment-15537141
 ] 

Sean Po commented on YARN-5384:
---

Test failure is known: 
https://issues.apache.org/jira/browse/YARN-5043?jql=text%20~%20%22testRMAppAttemptFailuresValidityInterval%22.

I also ran locally, and saw that the test ran successfully.

> Expose priority in ReservationSystem submission APIs
> 
>
> Key: YARN-5384
> URL: https://issues.apache.org/jira/browse/YARN-5384
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5384.v1.patch, YARN-5384.v10.patch, 
> YARN-5384.v11.patch, YARN-5384.v2.patch, YARN-5384.v3.patch, 
> YARN-5384.v4.patch, YARN-5384.v5.patch, YARN-5384.v6.patch, 
> YARN-5384.v7.patch, YARN-5384.v8.patch, YARN-5384.v9.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
> needed in ApplicationClientProtocol to accomplish it. Please refer to the 
> design doc in the parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5384) Expose priority in ReservationSystem submission APIs

2016-09-30 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5384?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5384:
--
Attachment: YARN-5384.v11.patch

YARN-5384.v11.patch removes the remaining checkstyle issue.

> Expose priority in ReservationSystem submission APIs
> 
>
> Key: YARN-5384
> URL: https://issues.apache.org/jira/browse/YARN-5384
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5384.v1.patch, YARN-5384.v10.patch, 
> YARN-5384.v11.patch, YARN-5384.v2.patch, YARN-5384.v3.patch, 
> YARN-5384.v4.patch, YARN-5384.v5.patch, YARN-5384.v6.patch, 
> YARN-5384.v7.patch, YARN-5384.v8.patch, YARN-5384.v9.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
> needed in ApplicationClientProtocol to accomplish it. Please refer to the 
> design doc in the parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Comment Edited] (YARN-5384) Expose priority in ReservationSystem submission APIs

2016-09-29 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5384?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15534925#comment-15534925
 ] 

Sean Po edited comment on YARN-5384 at 9/30/16 3:47 AM:


YARN-5384.v10.patch replaces all occurrences of the default 
Priority.newInstance(0) with Priority.UNDEFINED.


was (Author: seanpo03):
YARN-5384.v10.patch replaces all occurences of the default 
Priority.newInstance(0) with Priority.UNDEFINED.

> Expose priority in ReservationSystem submission APIs
> 
>
> Key: YARN-5384
> URL: https://issues.apache.org/jira/browse/YARN-5384
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5384.v1.patch, YARN-5384.v10.patch, 
> YARN-5384.v2.patch, YARN-5384.v3.patch, YARN-5384.v4.patch, 
> YARN-5384.v5.patch, YARN-5384.v6.patch, YARN-5384.v7.patch, 
> YARN-5384.v8.patch, YARN-5384.v9.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
> needed in ApplicationClientProtocol to accomplish it. Please refer to the 
> design doc in the parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5384) Expose priority in ReservationSystem submission APIs

2016-09-29 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5384?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5384:
--
Attachment: YARN-5384.v10.patch

YARN-5384.v10.patch replaces all occurences of the default 
Priority.newInstance(0) with Priority.UNDEFINED.

> Expose priority in ReservationSystem submission APIs
> 
>
> Key: YARN-5384
> URL: https://issues.apache.org/jira/browse/YARN-5384
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5384.v1.patch, YARN-5384.v10.patch, 
> YARN-5384.v2.patch, YARN-5384.v3.patch, YARN-5384.v4.patch, 
> YARN-5384.v5.patch, YARN-5384.v6.patch, YARN-5384.v7.patch, 
> YARN-5384.v8.patch, YARN-5384.v9.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
> needed in ApplicationClientProtocol to accomplish it. Please refer to the 
> design doc in the parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5384) Expose priority in ReservationSystem submission APIs

2016-09-29 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5384?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5384:
--
Attachment: YARN-5384.v9.patch

YARN-5384.v9.patch addresses all your comments [~subru]. Thanks!

> Expose priority in ReservationSystem submission APIs
> 
>
> Key: YARN-5384
> URL: https://issues.apache.org/jira/browse/YARN-5384
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5384.v1.patch, YARN-5384.v2.patch, 
> YARN-5384.v3.patch, YARN-5384.v4.patch, YARN-5384.v5.patch, 
> YARN-5384.v6.patch, YARN-5384.v7.patch, YARN-5384.v8.patch, YARN-5384.v9.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
> needed in ApplicationClientProtocol to accomplish it. Please refer to the 
> design doc in the parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5384) Expose priority in ReservationSystem submission APIs

2016-09-29 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5384?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5384:
--
Attachment: YARN-5384.v8.patch

YARN-5384.v8.patch removes checkstyle issues introduced by YARN-5384.v7.patch.

> Expose priority in ReservationSystem submission APIs
> 
>
> Key: YARN-5384
> URL: https://issues.apache.org/jira/browse/YARN-5384
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5384.v1.patch, YARN-5384.v2.patch, 
> YARN-5384.v3.patch, YARN-5384.v4.patch, YARN-5384.v5.patch, 
> YARN-5384.v6.patch, YARN-5384.v7.patch, YARN-5384.v8.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
> needed in ApplicationClientProtocol to accomplish it. Please refer to the 
> design doc in the parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5384) Expose priority in ReservationSystem submission APIs

2016-09-29 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5384?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5384:
--
Attachment: YARN-5384.v7.patch

YARN-5384.v7.patch fixes the test failures in TestGreedyReservationAgent.java.

> Expose priority in ReservationSystem submission APIs
> 
>
> Key: YARN-5384
> URL: https://issues.apache.org/jira/browse/YARN-5384
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5384.v1.patch, YARN-5384.v2.patch, 
> YARN-5384.v3.patch, YARN-5384.v4.patch, YARN-5384.v5.patch, 
> YARN-5384.v6.patch, YARN-5384.v7.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
> needed in ApplicationClientProtocol to accomplish it. Please refer to the 
> design doc in the parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5384) Expose priority in ReservationSystem submission APIs

2016-09-28 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5384?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5384:
--
Attachment: YARN-5384.v6.patch

YARN-5384.v6.patch fixes the checkstyle issues.

> Expose priority in ReservationSystem submission APIs
> 
>
> Key: YARN-5384
> URL: https://issues.apache.org/jira/browse/YARN-5384
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5384.v1.patch, YARN-5384.v2.patch, 
> YARN-5384.v3.patch, YARN-5384.v4.patch, YARN-5384.v5.patch, YARN-5384.v6.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
> needed in ApplicationClientProtocol to accomplish it. Please refer to the 
> design doc in the parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5384) Expose priority in ReservationSystem submission APIs

2016-09-28 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5384?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5384:
--
Attachment: YARN-5384.v5.patch

Thanks [~subru] for your comments! YARN-5384.v5.patch addresses all your 
comments, and removes some unrelated changes from YARN-5384.v4.patch,

> Expose priority in ReservationSystem submission APIs
> 
>
> Key: YARN-5384
> URL: https://issues.apache.org/jira/browse/YARN-5384
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5384.v1.patch, YARN-5384.v2.patch, 
> YARN-5384.v3.patch, YARN-5384.v4.patch, YARN-5384.v5.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
> needed in ApplicationClientProtocol to accomplish it. Please refer to the 
> design doc in the parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5384) Expose priority in ReservationSystem submission APIs

2016-09-28 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5384?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5384:
--
Attachment: YARN-5384.v4.patch

> Expose priority in ReservationSystem submission APIs
> 
>
> Key: YARN-5384
> URL: https://issues.apache.org/jira/browse/YARN-5384
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5384.v1.patch, YARN-5384.v2.patch, 
> YARN-5384.v3.patch, YARN-5384.v4.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
> needed in ApplicationClientProtocol to accomplish it. Please refer to the 
> design doc in the parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5384) Expose priority in ReservationSystem submission APIs

2016-09-27 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5384?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5384:
--
Attachment: YARN-5384.v3.patch

YARN-5384.v3.patch fixes the checkstyle issue with an unused import. However, 
ReservationDefinition.java does not have whitespace at line 184, nor was it 
introduced with YARN-5384.v2.patch of the patch so the issue is ignored for now.

> Expose priority in ReservationSystem submission APIs
> 
>
> Key: YARN-5384
> URL: https://issues.apache.org/jira/browse/YARN-5384
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5384.v1.patch, YARN-5384.v2.patch, 
> YARN-5384.v3.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
> needed in ApplicationClientProtocol to accomplish it. Please refer to the 
> design doc in the parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5385) Add a PriorityAgent in ReservationSystem

2016-09-27 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5385?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5385:
--
Attachment: YARN-5385.v1.patch

> Add a PriorityAgent in ReservationSystem 
> -
>
> Key: YARN-5385
> URL: https://issues.apache.org/jira/browse/YARN-5385
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5385.v1.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the addition 
> of a priority agent to accomplish it. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5384) Expose priority in ReservationSystem submission APIs

2016-09-27 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5384?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5384:
--
Attachment: YARN-5384.v2.patch

YARN-5384.v2.patch fixes the merge conflicts with latest trunk.

> Expose priority in ReservationSystem submission APIs
> 
>
> Key: YARN-5384
> URL: https://issues.apache.org/jira/browse/YARN-5384
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: capacity scheduler, fairscheduler, resourcemanager
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5384.v1.patch, YARN-5384.v2.patch
>
>
> YARN-5211 proposes adding support for generalized priorities for reservations 
> in the YARN ReservationSystem. This JIRA is a sub-task to track the changes 
> needed in ApplicationClientProtocol to accomplish it. Please refer to the 
> design doc in the parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5656) Fix ReservationACLsTestBase

2016-09-20 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15507677#comment-15507677
 ] 

Sean Po commented on YARN-5656:
---

Thanks for the review and commit [~asuresh]!

> Fix ReservationACLsTestBase
> ---
>
> Key: YARN-5656
> URL: https://issues.apache.org/jira/browse/YARN-5656
> Project: Hadoop YARN
>  Issue Type: Test
>Reporter: Sean Po
>Assignee: Sean Po
> Fix For: 2.9.0, 3.0.0-alpha2
>
> Attachments: YARN-5656.v1.patch, YARN-5656.v2.patch
>
>
> ReservationACLsTestBase fails when verifying that a reservation can be 
> successfully updated by a user who did not submit the reservation who also 
> has an admin ACL.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5656) ReservationACLsTestBase fails on trunk

2016-09-20 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15507475#comment-15507475
 ] 

Sean Po commented on YARN-5656:
---

The Javadoc failures are not caused by the latest patch. 

Changes were made to only four files: NoOverCommitPolicy.java, 
[MismatchedUserException.java], ReservationACLsTestBase.java and 
TestNoOverCommitPolicy.java.

None of these were referenced in the Javadoc failure results.

> ReservationACLsTestBase fails on trunk
> --
>
> Key: YARN-5656
> URL: https://issues.apache.org/jira/browse/YARN-5656
> Project: Hadoop YARN
>  Issue Type: Test
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5656.v1.patch, YARN-5656.v2.patch
>
>
> ReservationACLsTestBase fails when verifying that a reservation can be 
> successfully updated by a user who did not submit the reservation who also 
> has an admin ACL.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5656) ReservationACLsTestBase fails on trunk

2016-09-19 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5656?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5656:
--
Attachment: YARN-5656.v2.patch

Thanks [~asuresh] for the review. YARN-5656.v2.patch addresses your comments, 
and removes the unused MismatchedUserException.java file.

> ReservationACLsTestBase fails on trunk
> --
>
> Key: YARN-5656
> URL: https://issues.apache.org/jira/browse/YARN-5656
> Project: Hadoop YARN
>  Issue Type: Test
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5656.v1.patch, YARN-5656.v2.patch
>
>
> ReservationACLsTestBase fails when verifying that a reservation can be 
> successfully updated by a user who did not submit the reservation who also 
> has an admin ACL.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5656) ReservationACLsTestBase fails on trunk

2016-09-16 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15497412#comment-15497412
 ] 

Sean Po commented on YARN-5656:
---

I think it is safe to remove the check entirely given the two scenarios:

1. If ACLS are enabled, then a check that ensures that the reservation is being 
updated by an admin / by the user who submitted the reservation is redundant. 

2. Otherwise, if ACLs are not enabled, we are saying that anyone can update any 
reservation, so the check would be invalid.

> ReservationACLsTestBase fails on trunk
> --
>
> Key: YARN-5656
> URL: https://issues.apache.org/jira/browse/YARN-5656
> Project: Hadoop YARN
>  Issue Type: Test
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5656.v1.patch
>
>
> ReservationACLsTestBase fails when verifying that a reservation can be 
> successfully updated by a user who did not submit the reservation who also 
> has an admin ACL.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5656) ReservationACLsTestBase fails on trunk

2016-09-16 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5656?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5656:
--
Attachment: YARN-5656.v1.patch

YARN-5656.v1.patch removes  the check to verify that the same user is editing 
the reservation in the NoOverCommitPolicy. The patch also specifies the 
DominantResourceFairness policy in the ReservationACLsTestBase.

> ReservationACLsTestBase fails on trunk
> --
>
> Key: YARN-5656
> URL: https://issues.apache.org/jira/browse/YARN-5656
> Project: Hadoop YARN
>  Issue Type: Test
>Reporter: Sean Po
> Attachments: YARN-5656.v1.patch
>
>
> ReservationACLsTestBase fails when verifying that a reservation can be 
> successfully updated by a user who did not submit the reservation who also 
> has an admin ACL.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Created] (YARN-5656) ReservationACLsTestBase fails on trunk

2016-09-16 Thread Sean Po (JIRA)
Sean Po created YARN-5656:
-

 Summary: ReservationACLsTestBase fails on trunk
 Key: YARN-5656
 URL: https://issues.apache.org/jira/browse/YARN-5656
 Project: Hadoop YARN
  Issue Type: Test
Reporter: Sean Po


ReservationACLsTestBase fails when verifying that a reservation can be 
successfully updated by a user who did not submit the reservation who also has 
an admin ACL.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5331) Extend RLESparseResourceAllocation with period for supporting recurring reservations in YARN ReservationSystem

2016-09-06 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5331?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15468984#comment-15468984
 ] 

Sean Po commented on YARN-5331:
---

Thanks for the quick update [~ajsangeetha]! A few more minor comments:

* NIT: PeriodicRLESparseResourceAllocation.java+32
** Please put a javadoc for the constructor, and make it clear that timePeriod 
is in ms.
* PeriodicRLESparseResourceAllocation.java+68 
** I noticed you changed the comparison operator from <= to <. I think <= is 
actually correct. The reason is because addInterval seems to set the tick at 
endTime = ZERO_RESOURCE before adding to existing RLE. *Currently*, if 
interval.getEndTime() == timePeriod - 1 (which is the latest acceptable 
endTime), then RLE at tick timePeriod -1 will be set to ZERO_RESOURCE. I 
believe this is a bug.
** Please add a unit test to verify this logic. A simple case would be to 
setCapacityInInterval with interval from 0 to the latest acceptable endTime, 
and make sure that getCapacityAtTime(timePeriod - 1) is non zero. I believe 
this test case will fail with current code.
* NIT: PeriodicRLESparseResourceAllocation.java+83
** Similar checks from PeriodicRLESparseResourceAllocation.java+68 should be 
made here.
* TestPeriodicRLESparseResourceAllocation.java
** Since we have checks in 
PeriodicRLESparseResourceAllocation::setCapacityInInterval and 
PeriodicRLESparseResourceAllocation::removeInterval for valid time intervals, 
lets add positive and negative unit tests for these.




> Extend RLESparseResourceAllocation with period for supporting recurring 
> reservations in YARN ReservationSystem
> --
>
> Key: YARN-5331
> URL: https://issues.apache.org/jira/browse/YARN-5331
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Subru Krishnan
>Assignee: Sangeetha Abdu Jyothi
> Attachments: YARN-5331.001.patch, YARN-5331.002.patch, 
> YARN-5331.003.patch
>
>
> YARN-5326 proposes adding native support for recurring reservations in the 
> YARN ReservationSystem. This JIRA is a sub-task to add a 
> PeriodicRLESparseResourceAllocation. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5331) Extend RLESparseResourceAllocation with period for supporting recurring reservations in YARN ReservationSystem

2016-09-06 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5331?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15468705#comment-15468705
 ] 

Sean Po commented on YARN-5331:
---

Thanks [~ajsangeetha] for the patch! This looks good, except for a few very 
minor comments:
* NIT: PeriodicRLESParseResourceAllocation.java+27
** Remove extra newline.
* NIT: TestPeriodicRLESparseResourceAllocation.java+18 
** Add a newline.
* NIT: PeriodicRLESParseResourceAllocation.java+57
** It looks like RLESparseResourceAllocation.AddInterval would include 0, and 
does not include interval.getEndTime(). Please update the javadoc.
* NIT: PeriodicRLESParseResourceAllocation.java+61
** Extra space before "specified".
* PeriodicRLESParseResourceAllocation.java+65
** Do we need to check interval.getStartTime() >= 0 and interval.getEndTime() > 
interval.getStartTime()?
* TestPeriodicRLESparseResourceAllocation.java+81
** Can be static. 
** Also looks a lot like ReservationSystemTestUtil::generateAllocation. Maybe 
you can use that method instead, or move this one into 
ReservationSystemTestUtil. That way you can re-use in 
TestRLESparseResourceAllocation as well.
* TestRLESparseResourceAllocation.java+528
** Please include tests for this where tick > lastKey. Is it expected to be 
ZERO_RESOURCE?
** Also include test for tick == lastKey.

Question about PeriodicRLESparseResourceAllocation::setTimePeriod:
* If the new period is smaller than the previous period, should the behavior be 
to trim rleVector? It looks like there will be a bug for 
PeriodicRLESparseResourceAllocation::getMaxPeriodicCapacity if rleVector isn't 
trimmed, because the end condition is t <= lastKey.
** Please add a unit test for this condition.


> Extend RLESparseResourceAllocation with period for supporting recurring 
> reservations in YARN ReservationSystem
> --
>
> Key: YARN-5331
> URL: https://issues.apache.org/jira/browse/YARN-5331
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Subru Krishnan
>Assignee: Sangeetha Abdu Jyothi
> Attachments: YARN-5331.001.patch, YARN-5331.002.patch
>
>
> YARN-5326 proposes adding native support for recurring reservations in the 
> YARN ReservationSystem. This JIRA is a sub-task to add a 
> PeriodicRLESparseResourceAllocation. Please refer to the design doc in the 
> parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5560) Clean up bad exception catching practices in TestYarnClient

2016-08-29 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15446434#comment-15446434
 ] 

Sean Po commented on YARN-5560:
---

Thanks [~jlowe] for the review and committing!

> Clean up bad exception catching practices in TestYarnClient
> ---
>
> Key: YARN-5560
> URL: https://issues.apache.org/jira/browse/YARN-5560
> Project: Hadoop YARN
>  Issue Type: Test
>Reporter: Sean Po
>Assignee: Sean Po
> Fix For: 2.8.0
>
> Attachments: YARN-5560.v1.patch, YARN-5560.v2.patch, 
> YARN-5560.v3.patch
>
>
> In TestYarnClient, tests commonly wrap methods that throw exceptions in a try 
> catch statement similar to the following:
> {code}
> try {
> client.submitApplication(context);
> } catch (Exception e) {
> Assert.fail("Exception is not expected.");
> }
> {code}
> This hides useful error messages, and surfaces less helpful ones.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5560) Clean up bad exception catching practices in TestYarnClient

2016-08-26 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5560?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5560:
--
Attachment: YARN-5560.v3.patch

You're welcome [~jlowe], thanks for reviewing! V3 of the patch addresses your 
previous comment.

> Clean up bad exception catching practices in TestYarnClient
> ---
>
> Key: YARN-5560
> URL: https://issues.apache.org/jira/browse/YARN-5560
> Project: Hadoop YARN
>  Issue Type: Test
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5560.v1.patch, YARN-5560.v2.patch, 
> YARN-5560.v3.patch
>
>
> In TestYarnClient, tests commonly wrap methods that throw exceptions in a try 
> catch statement similar to the following:
> {code}
> try {
> client.submitApplication(context);
> } catch (Exception e) {
> Assert.fail("Exception is not expected.");
> }
> {code}
> This hides useful error messages, and surfaces less helpful ones.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5560) Clean up bad exception catching practices in TestYarnClient

2016-08-25 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5560?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5560:
--
Attachment: YARN-5560.v2.patch

V2 of the patch fixes the open checkstyle issue.

> Clean up bad exception catching practices in TestYarnClient
> ---
>
> Key: YARN-5560
> URL: https://issues.apache.org/jira/browse/YARN-5560
> Project: Hadoop YARN
>  Issue Type: Test
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5560.v1.patch, YARN-5560.v2.patch
>
>
> In TestYarnClient, tests commonly wrap methods that throw exceptions in a try 
> catch statement similar to the following:
> {code}
> try {
> client.submitApplication(context);
> } catch (Exception e) {
> Assert.fail("Exception is not expected.");
> }
> {code}
> This hides useful error messages, and surfaces less helpful ones.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5560) Clean up bad exception catching practices in TestYarnClient

2016-08-25 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5560?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5560:
--
Attachment: YARN-5560.v1.patch

First patch removes occurrences of catch blocks that only invoke Assert.fail.

> Clean up bad exception catching practices in TestYarnClient
> ---
>
> Key: YARN-5560
> URL: https://issues.apache.org/jira/browse/YARN-5560
> Project: Hadoop YARN
>  Issue Type: Test
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5560.v1.patch
>
>
> In TestYarnClient, tests commonly wrap methods that throw exceptions in a try 
> catch statement similar to the following:
> {code}
> try {
> client.submitApplication(context);
> } catch (Exception e) {
> Assert.fail("Exception is not expected.");
> }
> {code}
> This hides useful error messages, and surfaces less helpful ones.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Comment Edited] (YARN-5560) Clean up bad exception catching practices in TestYarnClient

2016-08-25 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15437460#comment-15437460
 ] 

Sean Po edited comment on YARN-5560 at 8/25/16 6:59 PM:


First patch removes occurrences of catch blocks that only invoke Assert.fail in 
TestYarnClient.


was (Author: seanpo03):
First patch removes occurrences of catch blocks that only invoke Assert.fail.

> Clean up bad exception catching practices in TestYarnClient
> ---
>
> Key: YARN-5560
> URL: https://issues.apache.org/jira/browse/YARN-5560
> Project: Hadoop YARN
>  Issue Type: Test
>Reporter: Sean Po
>Assignee: Sean Po
> Attachments: YARN-5560.v1.patch
>
>
> In TestYarnClient, tests commonly wrap methods that throw exceptions in a try 
> catch statement similar to the following:
> {code}
> try {
> client.submitApplication(context);
> } catch (Exception e) {
> Assert.fail("Exception is not expected.");
> }
> {code}
> This hides useful error messages, and surfaces less helpful ones.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5389) TestYarnClient#testReservationDelete fails

2016-08-25 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5389?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15437252#comment-15437252
 ] 

Sean Po commented on YARN-5389:
---

Thanks for the review [~jlowe]!

> TestYarnClient#testReservationDelete fails
> --
>
> Key: YARN-5389
> URL: https://issues.apache.org/jira/browse/YARN-5389
> Project: Hadoop YARN
>  Issue Type: Test
>Reporter: Rohith Sharma K S
>Assignee: Sean Po
>  Labels: test
> Fix For: 2.8.0
>
> Attachments: YARN-5389.v1.patch, YARN-5389.v2.patch, 
> YARN-5389.v3.patch, YARN-5389.v4.patch
>
>
> In build report 
> [report|https://builds.apache.org/job/PreCommit-YARN-Build/12341/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt],
>  below test fails. 
> {noformat}
> Tests run: 28, Failures: 3, Errors: 0, Skipped: 0, Time elapsed: 26.066 sec 
> <<< FAILURE! - in org.apache.hadoop.yarn.client.api.impl.TestYarnClient
> testReservationDelete(org.apache.hadoop.yarn.client.api.impl.TestYarnClient)  
> Time elapsed: 2.213 sec  <<< FAILURE!
> java.lang.AssertionError: Exhausted attempts in checking if node capacity was 
> added to the plan
>   at org.junit.Assert.fail(Assert.java:88)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.setupMiniYARNCluster(TestYarnClient.java:1222)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.testReservationDelete(TestYarnClient.java:1584)
> testListReservationsByInvalidTimeInterval(org.apache.hadoop.yarn.client.api.impl.TestYarnClient)
>   Time elapsed: 2.215 sec  <<< FAILURE!
> java.lang.AssertionError: Exhausted attempts in checking if node capacity was 
> added to the plan
>   at org.junit.Assert.fail(Assert.java:88)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.setupMiniYARNCluster(TestYarnClient.java:1222)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.testListReservationsByInvalidTimeInterval(TestYarnClient.java:1444)
> testListReservationsByTimeIntervalContainingNoReservations(org.apache.hadoop.yarn.client.api.impl.TestYarnClient)
>   Time elapsed: 2.206 sec  <<< FAILURE!
> java.lang.AssertionError: Exhausted attempts in checking if node capacity was 
> added to the plan
>   at org.junit.Assert.fail(Assert.java:88)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.setupMiniYARNCluster(TestYarnClient.java:1222)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.testListReservationsByTimeIntervalContainingNoReservations(TestYarnClient.java:1494)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5389) TestYarnClient#testReservationDelete fails in trunk

2016-08-24 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5389?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5389:
--
Attachment: YARN-5389.v4.patch

YARN-5389.v4.patch contains the changes spoken about in my previous comment.

> TestYarnClient#testReservationDelete fails in trunk
> ---
>
> Key: YARN-5389
> URL: https://issues.apache.org/jira/browse/YARN-5389
> Project: Hadoop YARN
>  Issue Type: Test
>Reporter: Rohith Sharma K S
>Assignee: Sean Po
>  Labels: test
> Attachments: YARN-5389.v1.patch, YARN-5389.v2.patch, 
> YARN-5389.v3.patch, YARN-5389.v4.patch
>
>
> In build report 
> [report|https://builds.apache.org/job/PreCommit-YARN-Build/12341/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt],
>  below test fails. 
> {noformat}
> Tests run: 28, Failures: 3, Errors: 0, Skipped: 0, Time elapsed: 26.066 sec 
> <<< FAILURE! - in org.apache.hadoop.yarn.client.api.impl.TestYarnClient
> testReservationDelete(org.apache.hadoop.yarn.client.api.impl.TestYarnClient)  
> Time elapsed: 2.213 sec  <<< FAILURE!
> java.lang.AssertionError: Exhausted attempts in checking if node capacity was 
> added to the plan
>   at org.junit.Assert.fail(Assert.java:88)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.setupMiniYARNCluster(TestYarnClient.java:1222)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.testReservationDelete(TestYarnClient.java:1584)
> testListReservationsByInvalidTimeInterval(org.apache.hadoop.yarn.client.api.impl.TestYarnClient)
>   Time elapsed: 2.215 sec  <<< FAILURE!
> java.lang.AssertionError: Exhausted attempts in checking if node capacity was 
> added to the plan
>   at org.junit.Assert.fail(Assert.java:88)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.setupMiniYARNCluster(TestYarnClient.java:1222)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.testListReservationsByInvalidTimeInterval(TestYarnClient.java:1444)
> testListReservationsByTimeIntervalContainingNoReservations(org.apache.hadoop.yarn.client.api.impl.TestYarnClient)
>   Time elapsed: 2.206 sec  <<< FAILURE!
> java.lang.AssertionError: Exhausted attempts in checking if node capacity was 
> added to the plan
>   at org.junit.Assert.fail(Assert.java:88)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.setupMiniYARNCluster(TestYarnClient.java:1222)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.testListReservationsByTimeIntervalContainingNoReservations(TestYarnClient.java:1494)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5389) TestYarnClient#testReservationDelete fails in trunk

2016-08-24 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5389?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15435907#comment-15435907
 ] 

Sean Po commented on YARN-5389:
---

Thanks for the comments [~jlowe]. I agree - in the next patch, I will revert 
changes that remove code that does something similar to the following:

{code}
try {
  sResponse = client.submitReservation(sRequest);
} catch (Exception e) {
  Assert.fail(e.getMessage());
}
{code}

I've created YARN-5560 to track cleaning up such patterns in TestYarnClient.

> TestYarnClient#testReservationDelete fails in trunk
> ---
>
> Key: YARN-5389
> URL: https://issues.apache.org/jira/browse/YARN-5389
> Project: Hadoop YARN
>  Issue Type: Test
>Reporter: Rohith Sharma K S
>Assignee: Sean Po
>  Labels: test
> Attachments: YARN-5389.v1.patch, YARN-5389.v2.patch, 
> YARN-5389.v3.patch
>
>
> In build report 
> [report|https://builds.apache.org/job/PreCommit-YARN-Build/12341/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt],
>  below test fails. 
> {noformat}
> Tests run: 28, Failures: 3, Errors: 0, Skipped: 0, Time elapsed: 26.066 sec 
> <<< FAILURE! - in org.apache.hadoop.yarn.client.api.impl.TestYarnClient
> testReservationDelete(org.apache.hadoop.yarn.client.api.impl.TestYarnClient)  
> Time elapsed: 2.213 sec  <<< FAILURE!
> java.lang.AssertionError: Exhausted attempts in checking if node capacity was 
> added to the plan
>   at org.junit.Assert.fail(Assert.java:88)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.setupMiniYARNCluster(TestYarnClient.java:1222)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.testReservationDelete(TestYarnClient.java:1584)
> testListReservationsByInvalidTimeInterval(org.apache.hadoop.yarn.client.api.impl.TestYarnClient)
>   Time elapsed: 2.215 sec  <<< FAILURE!
> java.lang.AssertionError: Exhausted attempts in checking if node capacity was 
> added to the plan
>   at org.junit.Assert.fail(Assert.java:88)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.setupMiniYARNCluster(TestYarnClient.java:1222)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.testListReservationsByInvalidTimeInterval(TestYarnClient.java:1444)
> testListReservationsByTimeIntervalContainingNoReservations(org.apache.hadoop.yarn.client.api.impl.TestYarnClient)
>   Time elapsed: 2.206 sec  <<< FAILURE!
> java.lang.AssertionError: Exhausted attempts in checking if node capacity was 
> added to the plan
>   at org.junit.Assert.fail(Assert.java:88)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.setupMiniYARNCluster(TestYarnClient.java:1222)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.testListReservationsByTimeIntervalContainingNoReservations(TestYarnClient.java:1494)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5560) Clean up bad exception catching practices in TestYarnClient

2016-08-24 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5560?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5560:
--
Description: 
In TestYarnClient, tests commonly wrap methods that throw exceptions in a try 
catch statement similar to the following:


{code}
try {
client.submitApplication(context);
} catch (Exception e) {
Assert.fail("Exception is not expected.");
}
{code}

This hides useful error messages, and surfaces less helpful ones.

  was:
In TestYarnClient, tests commonly wrap methods that throw exceptions in a try 
catch statement similar to the following:

{{try {}}
{{client.submitApplication(context);}}
{{} catch (Exception e) {}}
{{Assert.fail("Exception is not expected.");}}
{{}}}

This hides useful error messages, and surfaces less helpful ones.


> Clean up bad exception catching practices in TestYarnClient
> ---
>
> Key: YARN-5560
> URL: https://issues.apache.org/jira/browse/YARN-5560
> Project: Hadoop YARN
>  Issue Type: Test
>Reporter: Sean Po
>Assignee: Sean Po
>
> In TestYarnClient, tests commonly wrap methods that throw exceptions in a try 
> catch statement similar to the following:
> {code}
> try {
> client.submitApplication(context);
> } catch (Exception e) {
> Assert.fail("Exception is not expected.");
> }
> {code}
> This hides useful error messages, and surfaces less helpful ones.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Created] (YARN-5560) Clean up bad exception catching practices in TestYarnClient

2016-08-24 Thread Sean Po (JIRA)
Sean Po created YARN-5560:
-

 Summary: Clean up bad exception catching practices in 
TestYarnClient
 Key: YARN-5560
 URL: https://issues.apache.org/jira/browse/YARN-5560
 Project: Hadoop YARN
  Issue Type: Test
Reporter: Sean Po
Assignee: Sean Po


In TestYarnClient, tests commonly wrap methods that throw exceptions in a try 
catch statement similar to the following:

{{try {}}
{{client.submitApplication(context);}}
{{} catch (Exception e) {}}
{{Assert.fail("Exception is not expected.");}}
{{}}}

This hides useful error messages, and surfaces less helpful ones.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Updated] (YARN-5389) TestYarnClient#testReservationDelete fails in trunk

2016-08-24 Thread Sean Po (JIRA)

 [ 
https://issues.apache.org/jira/browse/YARN-5389?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Po updated YARN-5389:
--
Attachment: YARN-5389.v3.patch

Thanks [~jlowe], the 10ms suggestion was overlooked in your previous 
suggestion, but the change is made in v3 of the patch. I also removed any try 
catch that asserts fail in the event of an exception to let the exception 
bubble up.

> TestYarnClient#testReservationDelete fails in trunk
> ---
>
> Key: YARN-5389
> URL: https://issues.apache.org/jira/browse/YARN-5389
> Project: Hadoop YARN
>  Issue Type: Test
>Reporter: Rohith Sharma K S
>Assignee: Sean Po
>  Labels: test
> Attachments: YARN-5389.v1.patch, YARN-5389.v2.patch, 
> YARN-5389.v3.patch
>
>
> In build report 
> [report|https://builds.apache.org/job/PreCommit-YARN-Build/12341/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt],
>  below test fails. 
> {noformat}
> Tests run: 28, Failures: 3, Errors: 0, Skipped: 0, Time elapsed: 26.066 sec 
> <<< FAILURE! - in org.apache.hadoop.yarn.client.api.impl.TestYarnClient
> testReservationDelete(org.apache.hadoop.yarn.client.api.impl.TestYarnClient)  
> Time elapsed: 2.213 sec  <<< FAILURE!
> java.lang.AssertionError: Exhausted attempts in checking if node capacity was 
> added to the plan
>   at org.junit.Assert.fail(Assert.java:88)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.setupMiniYARNCluster(TestYarnClient.java:1222)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.testReservationDelete(TestYarnClient.java:1584)
> testListReservationsByInvalidTimeInterval(org.apache.hadoop.yarn.client.api.impl.TestYarnClient)
>   Time elapsed: 2.215 sec  <<< FAILURE!
> java.lang.AssertionError: Exhausted attempts in checking if node capacity was 
> added to the plan
>   at org.junit.Assert.fail(Assert.java:88)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.setupMiniYARNCluster(TestYarnClient.java:1222)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.testListReservationsByInvalidTimeInterval(TestYarnClient.java:1444)
> testListReservationsByTimeIntervalContainingNoReservations(org.apache.hadoop.yarn.client.api.impl.TestYarnClient)
>   Time elapsed: 2.206 sec  <<< FAILURE!
> java.lang.AssertionError: Exhausted attempts in checking if node capacity was 
> added to the plan
>   at org.junit.Assert.fail(Assert.java:88)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.setupMiniYARNCluster(TestYarnClient.java:1222)
>   at 
> org.apache.hadoop.yarn.client.api.impl.TestYarnClient.testListReservationsByTimeIntervalContainingNoReservations(TestYarnClient.java:1494)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



[jira] [Commented] (YARN-5327) API changes required to support recurring reservations in the YARN ReservationSystem

2016-08-23 Thread Sean Po (JIRA)

[ 
https://issues.apache.org/jira/browse/YARN-5327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15433994#comment-15433994
 ] 

Sean Po commented on YARN-5327:
---

Thanks for addressing the comments [~ajsangeetha], this patch looks good to me. 
+1 after the checkstyle issues have been resolved.

> API changes required to support recurring reservations in the YARN 
> ReservationSystem
> 
>
> Key: YARN-5327
> URL: https://issues.apache.org/jira/browse/YARN-5327
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: resourcemanager
>Reporter: Subru Krishnan
>Assignee: Sangeetha Abdu Jyothi
> Attachments: YARN-5327.001.patch, YARN-5327.002.patch, 
> YARN-5327.003.patch, YARN-5327.004.patch, YARN-5327.005.patch
>
>
> YARN-5326 proposes adding native support for recurring reservations in the 
> YARN ReservationSystem. This JIRA is a sub-task to track the changes needed 
> in ApplicationClientProtocol to accomplish it. Please refer to the design doc 
> in the parent JIRA for details.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org



  1   2   3   >