hadoop-common-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From aajis...@apache.org
Subject hadoop git commit: YARN-6379. Remove unused argument in ClientRMService. Contributed by Kai Sasaki.
Date Mon, 27 Mar 2017 02:26:59 GMT
Repository: hadoop
Updated Branches:
  refs/heads/branch-2 f51ee482a -> e4320305d


YARN-6379. Remove unused argument in ClientRMService. Contributed by Kai Sasaki.

(cherry picked from commit b63ae854f9f5a900ca860dcb26c30785af32bde0)


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

Branch: refs/heads/branch-2
Commit: e4320305d3f9b2d85fd9e6079a72b9418a920cfa
Parents: f51ee48
Author: Akira Ajisaka <aajisaka@apache.org>
Authored: Mon Mar 27 11:23:41 2017 +0900
Committer: Akira Ajisaka <aajisaka@apache.org>
Committed: Mon Mar 27 11:26:23 2017 +0900

----------------------------------------------------------------------
 .../yarn/server/resourcemanager/ClientRMService.java   | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/e4320305/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 3ee13eb..aeacd2e 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
@@ -1155,7 +1155,7 @@ public class ClientRMService extends AbstractService implements
   @Override
   public GetNewReservationResponse getNewReservation(
       GetNewReservationRequest request) throws YarnException, IOException {
-    checkReservationSystem(AuditConstants.CREATE_NEW_RESERVATION_REQUEST);
+    checkReservationSystem();
     GetNewReservationResponse response =
         recordFactory.newRecordInstance(GetNewReservationResponse.class);
 
@@ -1169,7 +1169,7 @@ public class ClientRMService extends AbstractService implements
   public ReservationSubmissionResponse submitReservation(
       ReservationSubmissionRequest request) throws YarnException, IOException {
     // Check if reservation system is enabled
-    checkReservationSystem(AuditConstants.SUBMIT_RESERVATION_REQUEST);
+    checkReservationSystem();
     ReservationSubmissionResponse response =
         recordFactory.newRecordInstance(ReservationSubmissionResponse.class);
     ReservationId reservationId = request.getReservationId();
@@ -1228,7 +1228,7 @@ public class ClientRMService extends AbstractService implements
   public ReservationUpdateResponse updateReservation(
       ReservationUpdateRequest request) throws YarnException, IOException {
     // Check if reservation system is enabled
-    checkReservationSystem(AuditConstants.UPDATE_RESERVATION_REQUEST);
+    checkReservationSystem();
     ReservationUpdateResponse response =
         recordFactory.newRecordInstance(ReservationUpdateResponse.class);
     // Validate the input
@@ -1267,7 +1267,7 @@ public class ClientRMService extends AbstractService implements
   public ReservationDeleteResponse deleteReservation(
       ReservationDeleteRequest request) throws YarnException, IOException {
     // Check if reservation system is enabled
-    checkReservationSystem(AuditConstants.DELETE_RESERVATION_REQUEST);
+    checkReservationSystem();
     ReservationDeleteResponse response =
         recordFactory.newRecordInstance(ReservationDeleteResponse.class);
     // Validate the input
@@ -1306,7 +1306,7 @@ public class ClientRMService extends AbstractService implements
   public ReservationListResponse listReservations(
         ReservationListRequest requestInfo) throws YarnException, IOException {
     // Check if reservation system is enabled
-    checkReservationSystem(AuditConstants.LIST_RESERVATION_REQUEST);
+    checkReservationSystem();
     ReservationListResponse response =
             recordFactory.newRecordInstance(ReservationListResponse.class);
 
@@ -1381,7 +1381,8 @@ public class ClientRMService extends AbstractService implements
     return response;
   }
 
-  private void checkReservationSystem(String auditConstant) throws YarnException {
+  private void checkReservationSystem()
+      throws YarnException {
     // Check if reservation is enabled
     if (reservationSystem == null) {
       throw RPCUtil.getRemoteException("Reservation is not enabled."


---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org


Mime
View raw message