diff --git a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketAclRequest.java b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketAclRequest.java index f1a504f185bd..167d012aab5c 100644 --- a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketAclRequest.java +++ b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketAclRequest.java @@ -33,6 +33,7 @@ import org.apache.hadoop.ozone.om.helpers.OmBucketInfo; import org.apache.hadoop.ozone.om.ratis.utils.OzoneManagerDoubleBufferHelper; import org.apache.hadoop.ozone.om.request.OMClientRequest; +import org.apache.hadoop.ozone.om.response.bucket.acl.OMBucketAclResponse; import org.apache.hadoop.ozone.util.BooleanBiFunction; import org.apache.hadoop.ozone.om.request.util.ObjectParser; import org.apache.hadoop.ozone.om.response.OMClientResponse; @@ -204,8 +205,11 @@ abstract OMClientResponse onSuccess( * @param exception * @return OMClientResponse */ - abstract OMClientResponse onFailure(OMResponse.Builder omResponse, - IOException exception); + OMClientResponse onFailure(OMResponse.Builder omResponse, + IOException exception) { + return new OMBucketAclResponse( + createErrorOMResponse(omResponse, exception)); + } /** * Completion hook for final processing before return without lock. diff --git a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketAddAclRequest.java b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketAddAclRequest.java index e0cddf337207..e6e4bc528e93 100644 --- a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketAddAclRequest.java +++ b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketAddAclRequest.java @@ -119,13 +119,6 @@ OMClientResponse onSuccess(OMResponse.Builder omResponse, return new OMBucketAclResponse(omResponse.build(), omBucketInfo); } - @Override - OMClientResponse onFailure(OMResponse.Builder omResponse, - IOException exception) { - return new OMBucketAclResponse( - createErrorOMResponse(omResponse, exception)); - } - @Override void onComplete(boolean operationResult, IOException exception, OMMetrics omMetrics, AuditLogger auditLogger, diff --git a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketRemoveAclRequest.java b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketRemoveAclRequest.java index c1fa3f9e1e8b..52e65ab1da73 100644 --- a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketRemoveAclRequest.java +++ b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketRemoveAclRequest.java @@ -116,13 +116,6 @@ OMClientResponse onSuccess(OMResponse.Builder omResponse, return new OMBucketAclResponse(omResponse.build(), omBucketInfo); } - @Override - OMClientResponse onFailure(OMResponse.Builder omResponse, - IOException exception) { - return new OMBucketAclResponse( - createErrorOMResponse(omResponse, exception)); - } - @Override void onComplete(boolean operationResult, IOException exception, OMMetrics omMetrics, AuditLogger auditLogger, diff --git a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketSetAclRequest.java b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketSetAclRequest.java index 8cfe29ccd042..d4ba01750575 100644 --- a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketSetAclRequest.java +++ b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/acl/OMBucketSetAclRequest.java @@ -117,13 +117,6 @@ OMClientResponse onSuccess(OMResponse.Builder omResponse, return new OMBucketAclResponse(omResponse.build(), omBucketInfo); } - @Override - OMClientResponse onFailure(OMResponse.Builder omResponse, - IOException exception) { - return new OMBucketAclResponse( - createErrorOMResponse(omResponse, exception)); - } - @Override void onComplete(boolean operationResult, IOException exception, OMMetrics omMetrics, AuditLogger auditLogger,