HDDS-11586. Remove duplicate OzoneManagerProtocolServerSideTranslator…

…PB#OM_REQUESTS_PACKAGE
apache · Oct 15, 2024 · d6eb87a · d6eb87a
1 parent 2139367
commit d6eb87a
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/...n/java/org/apache/hadoop/ozone/protocolPB/OzoneManagerProtocolServerSideTranslatorPB.java b/...n/java/org/apache/hadoop/ozone/protocolPB/OzoneManagerProtocolServerSideTranslatorPB.java
@@ -69,7 +69,6 @@
  */
 public class OzoneManagerProtocolServerSideTranslatorPB implements OzoneManagerProtocolPB {
   private static final Logger LOG = LoggerFactory .getLogger(OzoneManagerProtocolServerSideTranslatorPB.class);
-  private static final String OM_REQUESTS_PACKAGE = "org.apache.hadoop.ozone";
 
   private final OzoneManagerRatisServer omRatisServer;
   private final RequestHandler handler;
@@ -123,7 +122,6 @@ public OzoneManagerProtocolServerSideTranslatorPB(
 
     // TODO: make this injectable for testing...
     this.requestValidations = new RequestValidations()
-        .fromPackage(OM_REQUESTS_PACKAGE)
         .withinContext(ValidationContext.of(ozoneManager.getVersionManager(), ozoneManager.getMetadataManager()))
         .load();
   }
@@ -214,8 +212,7 @@ private OMResponse internalProcessRequest(OMRequest request) throws
         //  return null, which triggered the findbugs warning.
         //  Added the assertion.
         assert (omClientRequest != null);
-        OMClientRequest finalOmClientRequest = omClientRequest;
-        requestToSubmit = preExecute(finalOmClientRequest);
+        requestToSubmit = preExecute(omClientRequest);
         this.lastRequestToSubmit = requestToSubmit;
       } catch (IOException ex) {
         if (omClientRequest != null) {