Repository: hadoop
Updated Branches:
  refs/heads/branch-2 9897538a9 -> a0dceec58


YARN-4825. Remove redundant code in ClientRMService::listReservations. (subru 
via asuresh)

(cherry picked from commit d82e797b652f248e238bdf1818e6b4a5b91cea7a)


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/098a6b8f
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/098a6b8f
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/098a6b8f

Branch: refs/heads/branch-2
Commit: 098a6b8f193fd58400161eab1aaa7b05b432eb3c
Parents: 9897538
Author: Arun Suresh <[email protected]>
Authored: Thu Mar 24 09:59:55 2016 -0700
Committer: Subru Krishnan <[email protected]>
Committed: Thu Oct 26 20:04:19 2017 -0700

----------------------------------------------------------------------
 .../hadoop/yarn/server/resourcemanager/ClientRMService.java | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/098a6b8f/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/ClientRMService.java
----------------------------------------------------------------------
diff --git 
a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/ClientRMService.java
 
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/ClientRMService.java
index 1fe2237..5b4123d 100644
--- 
a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/ClientRMService.java
+++ 
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/ClientRMService.java
@@ -1381,20 +1381,13 @@ public class ClientRMService extends AbstractService 
implements
     checkReservationACLs(requestInfo.getQueue(),
             AuditConstants.LIST_RESERVATION_REQUEST, reservationId);
 
-    ReservationId requestedId = null;
-    if (requestInfo.getReservationId() != null
-            && !requestInfo.getReservationId().isEmpty()) {
-      requestedId = ReservationId.parseReservationId(requestInfo
-            .getReservationId());
-    }
-
     long startTime = Math.max(requestInfo.getStartTime(), 0);
     long endTime = requestInfo.getEndTime() <= -1? Long.MAX_VALUE : requestInfo
             .getEndTime();
 
     Set<ReservationAllocation> reservations;
 
-    reservations = plan.getReservations(requestedId, new ReservationInterval(
+    reservations = plan.getReservations(reservationId, new ReservationInterval(
             startTime, endTime));
 
     List<ReservationAllocationState> info =


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to