aryangupta1998 opened a new pull request #2819:
URL: https://github.com/apache/ozone/pull/2819


   ## What changes were proposed in this pull request?
   
   Presently FILE_SYSTEM_OPTIMIZED("FSO") and OBJECT_STORE("OBS") 
implementation depends on "ozone.om.enable.filesystem.paths" flag. This flag 
should be enabled explicitly for the FSO logic and disabled explicitly for the 
OBS logic. This task is to remove the dependency with the normalization config 
flag for the FSO and OBS bucket layouts. 
   
   case-1) FSO bucket layout should normalize the given key path irrespective 
of the above flag value.
   
   case-2) OBS bucket layout shouldn't normalize the given key path 
irrespective of the above flag value.
   
   case-3) LEGACY bucket layout should read the configuration flag and perform 
normalize the given key path based on the above config flag.
   
    
   
   Hint
   
   `// normalize key if required.`
    `String keyName;`
   ` if (getBucketLayout() == BucketLayout.OBJECT_STORE) {`
         `keyName = keyArgs.getKeyName(); // pure object store semantics.`
    `} else if (getBucketLayout() == BucketLayout.FILE_SYSTEM_OPTIMIZED) {`
         `keyName = validateAndNormalizeKey(true, keyArgs.getKeyName()); // 
pure FS semantics.`
   ` } else {`
        ` // legacy bucket, normalize key based on 
"ozone.om.enable.filesystem.paths" flag value.`
         `keyName = validateAndNormalizeKey(`
        ` ozoneManager.getEnableFileSystemPaths(), keyArgs.getKeyName());`
   ` }`
   
    
   
   ## What is the link to the Apache JIRA
   
   https://issues.apache.org/jira/browse/HDDS-5929
   
   ## How was this patch tested?
   
   Tested Manually 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



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

Reply via email to