You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2020/08/13 17:33:15 UTC

[GitHub] [hadoop-ozone] bharatviswa504 opened a new pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

bharatviswa504 opened a new pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328


   ## What changes were proposed in this pull request?
   
   When ozone.om.enable.filesystem.paths, OM normalizes path, and stores the Keyname.
   
   Now when user tries to read the file from S3 using the keyName which user has used to create the Key, it will return error KEY_NOT_FOUND
   
   The issue is, lookupKey also need to normalize path, when ozone.om.enable.filesystem.paths is enabled. This is a common API used by S3/FS.
   
   ## What is the link to the Apache JIRA
   
   https://issues.apache.org/jira/browse/HDDS-4102
   
   ## How was this patch tested?
   
   Added a test.
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] bharatviswa504 commented on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
bharatviswa504 commented on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-681125960


   > Thanks for the patch @bharatviswa504, I would suggest adding a related robot test, too, which would make it possible to test the AWS compatibility of the change.
   > 
   > What do you think?
   
   Opened a jira to test S3 test suite with ozone.om.enable.filesystem.paths enabled.
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] bharatviswa504 edited a comment on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
bharatviswa504 edited a comment on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-684950217


   @elek 
   There is no issue with this, and I believe we have agreed on the part we need this like 100% HCFS with few changes to AWS S3 semantics. I don't see what is the problem in moving forward?. 
   
   As the other argument is when the flag is disabled to support compromised HCFS and 100% AWS.
   
   
   Let me know if you have any concerns about this PR.
   
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] bharatviswa504 edited a comment on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
bharatviswa504 edited a comment on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-681125960


   > Thanks for the patch @bharatviswa504, I would suggest adding a related robot test, too, which would make it possible to test the AWS compatibility of the change.
   > 
   > What do you think?
   
   Opened a jira HDDS-4154 to test S3 test suite with ozone.om.enable.filesystem.paths enabled.
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] bharatviswa504 commented on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
bharatviswa504 commented on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-692328313


   I have rebased this PR.
   
   cc @elek @arp7 


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] bharatviswa504 edited a comment on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
bharatviswa504 edited a comment on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-682070057


   > > Opened a jira [HDDS-4154](https://issues.apache.org/jira/browse/HDDS-4154) to test S3 test suite with ozone.om.enable.filesystem.paths enabled.
   > 
   > Thanks. I think we need additional s3 tests as the behavior what I tested above (and failing) is not part of the current test suite.
   > 
   > > Uploaded design doc has provided reasoning to perform on the OM side.
   > 
   > Thanks to share it. Now I understood the motivation. But what is your opinion about the AWS compatibility? For me it seems to be a bigger problem compared to having the validation code in one at the OM or one place at the client side.
   > 
   > What is your opinion about AWS compatibility issues?
   
   When this flag is enabled, as the config explains, we consider paths as unix paths and provide file system semantics. So when this is enabled we cannot gurantee 100% AWS compatibility. As said we will make this bucket level config to provide ease to users to have both on a cluster.
   
   > > Opened a jira [HDDS-4154](https://issues.apache.org/jira/browse/HDDS-4154) to test S3 test suite with ozone.om.enable.filesystem.paths enabled.
   > 
   > Thanks. I think we need additional s3 tests as the behavior what I tested above (and failing) is not part of the current test suite.
   > 
   > > Uploaded design doc has provided reasoning to perform on the OM side.
   > 
   > Thanks to share it. Now I understood the motivation. But what is your opinion about the AWS compatibility? For me it seems to be a bigger problem compared to having the validation code in one at the OM or one place at the client side.
   > 
   > What is your opinion about AWS compatibility issues?
   
   Currently, the test is simulated in the Integration test. As when this flag is enabled user has chosen to consider key as unix paths. I will add a test suite in HDDS-4154. For now, all tests are in integration tests.
   
   
   ```
     <property>
       <name>ozone.om.enable.filesystem.paths</name>
       <tag>OZONE, OM</tag>
       <value>false</value>
       <description>If true, key names will be interpreted as file system paths.
         "/" will be treated as a special character and paths will be normalized
         and must follow Unix filesystem path naming conventions. This flag will
         be helpful when objects created by S3G need to be accessed using OFS/O3Fs.
         If false, it will fallback to default behavior of Key/MPU create
         requests where key paths are not normalized and any intermediate
         directories will not be created or any file checks happens to check
         filesystem semantics.
       </description>
     </property>
   ```
   
   So, test/file/ will be normalized to test/file. This PR provides user ability to read with the same file name user has used during the creation of key "test/file/". 
   
   
   
   
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] elek commented on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
elek commented on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-681903253


   > Opened a jira HDDS-4154 to test S3 test suite with ozone.om.enable.filesystem.paths enabled.
   
   Thanks. I think we need additional s3 tests as the behavior what I tested above (and failing) is not part of the current test suite. 
   
   > Uploaded design doc has provided reasoning to perform on the OM side.
   
   Thanks to share it. Now I understood the motivation. But what is your opinion about the AWS compatibility? For me it seems to be a bigger problem compared to having the validation code in one at the OM or one place at the client side.
   
   What is your opinion about AWS compatibility issues?  
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] bharatviswa504 commented on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
bharatviswa504 commented on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-681124935


   > Tested and `file/` and `file` are different:
   > 
   > ```
   > $ aws s3api get-object --bucket=ozonetest --key=test1/file /tmp/www
   > {
   >     "AcceptRanges": "bytes",
   >     "LastModified": "Wed, 26 Aug 2020 14:24:06 GMT",
   >     "ContentLength": 79,
   >     "ETag": "\"6fcb0dbe8f0ef330b38f81a6a6a7a66e\"",
   >     "ContentType": "text/markdown",
   >     "Metadata": {}
   > }
   > ```
   > 
   > ```
   > aws s3api get-object --bucket=ozonetest --key=test1/file/ /tmp/www
   > 
   > An error occurred (NoSuchKey) when calling the GetObject operation: The specified key does not exist.
   > ```
   > 
   > Just an idea: what do you think about normalizing the path on the client side? (o3fs/ofs). Would it make any problems?
   
   Thank You @elek for the test.
   
   Uploaded design doc has provided reasoning to perform on the OM side.
   
   And also when used via ofs/o3fs, it uses path Object, and paths are normalized already.
   
   
   ![image](https://user-images.githubusercontent.com/8586345/91357044-75c9c900-e7a5-11ea-8eca-b056bf09bb3c.png)
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] elek commented on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
elek commented on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-680919139


   Tested and `file/` and `file` are different:
    
   ```
   $ aws s3api get-object --bucket=ozonetest --key=test1/file /tmp/www
   {
       "AcceptRanges": "bytes",
       "LastModified": "Wed, 26 Aug 2020 14:24:06 GMT",
       "ContentLength": 79,
       "ETag": "\"6fcb0dbe8f0ef330b38f81a6a6a7a66e\"",
       "ContentType": "text/markdown",
       "Metadata": {}
   }
   ```
   
   ```
   aws s3api get-object --bucket=ozonetest --key=test1/file/ /tmp/www
   
   An error occurred (NoSuchKey) when calling the GetObject operation: The specified key does not exist.
   ```
   
   Just an idea: what do you think about normalizing the path on the client side? (o3fs/ofs). Would it make any problems?


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] bharatviswa504 edited a comment on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
bharatviswa504 edited a comment on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-684950217


   @elek 
   There is no issue with this. As the other argument is when the flag is disabled to support HCFS.
   
   And I believe we have agreed on the part we need this like 100% HCFS with few changes to AWS S3 semantics. I don't see what is the problem in moving forward?
   
   Let me know if you have any concerns.
   
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] arp7 commented on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
arp7 commented on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-684929618


   > What is your opinion about AWS compatibility issues?
   
   What exactly is the compatibility issue? If the paths are not interpreted (default behavior), then there is full AWS compatibility.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] bharatviswa504 commented on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
bharatviswa504 commented on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-684950217


   @elek 
   There is no issue with this. As the other argument is when the flag is disabled to support HCFS.
   
   And I believe we have agreed on the part we need this like 100% HCFS with few changes to AWS S3 semantics.
   
   Let me know if you have any concerns.
   
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] bharatviswa504 commented on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
bharatviswa504 commented on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-682070057


   > > Opened a jira [HDDS-4154](https://issues.apache.org/jira/browse/HDDS-4154) to test S3 test suite with ozone.om.enable.filesystem.paths enabled.
   > 
   > Thanks. I think we need additional s3 tests as the behavior what I tested above (and failing) is not part of the current test suite.
   > 
   > > Uploaded design doc has provided reasoning to perform on the OM side.
   > 
   > Thanks to share it. Now I understood the motivation. But what is your opinion about the AWS compatibility? For me it seems to be a bigger problem compared to having the validation code in one at the OM or one place at the client side.
   > 
   > What is your opinion about AWS compatibility issues?
   
   When this flag is enabled, 333
   
   > > Opened a jira [HDDS-4154](https://issues.apache.org/jira/browse/HDDS-4154) to test S3 test suite with ozone.om.enable.filesystem.paths enabled.
   > 
   > Thanks. I think we need additional s3 tests as the behavior what I tested above (and failing) is not part of the current test suite.
   > 
   > > Uploaded design doc has provided reasoning to perform on the OM side.
   > 
   > Thanks to share it. Now I understood the motivation. But what is your opinion about the AWS compatibility? For me it seems to be a bigger problem compared to having the validation code in one at the OM or one place at the client side.
   > 
   > What is your opinion about AWS compatibility issues?
   
   Currently, the test is simulated in the Integration test. As when this flag is enabled user has chosen to consider key as unix paths. I will add a test suite in HDDS-4154. For now, all tests are in integration tests.
   
   
   ```
     <property>
       <name>ozone.om.enable.filesystem.paths</name>
       <tag>OZONE, OM</tag>
       <value>false</value>
       <description>If true, key names will be interpreted as file system paths.
         "/" will be treated as a special character and paths will be normalized
         and must follow Unix filesystem path naming conventions. This flag will
         be helpful when objects created by S3G need to be accessed using OFS/O3Fs.
         If false, it will fallback to default behavior of Key/MPU create
         requests where key paths are not normalized and any intermediate
         directories will not be created or any file checks happens to check
         filesystem semantics.
       </description>
     </property>
   ```
   
   So, test/file/ will be normalized to test/file. This PR provides user ability to read with the same file name user has used during the creation of key "test/file/". 
   
   
   
   
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] elek commented on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
elek commented on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-684923522


   > @elek Any more comments?
   
   Looks like we don't have the consensus, yet, in HDDS-4097. It was discussed yesterday during the community meeting, and I felt that everybody agree more or less, but I am not sure if @arp7 fully agrees.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] elek commented on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
elek commented on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-685703905


   I uploaded my proposal to https://issues.apache.org/jira/browse/HDDS-4097, please let me know your opinion.
   
   I think the normalization should be optional and should be driven by a configuration which is independent of the intermediate directory creation (`ozone.om.enable.filesystem.paths`).
   
   While intermediate directories are a must-have for providing services for both S3 and HCFS, normalization seems to be an optional (but very useful) feature.
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] bharatviswa504 edited a comment on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
bharatviswa504 edited a comment on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-681124935


   > Tested and `file/` and `file` are different:
   > 
   > ```
   > $ aws s3api get-object --bucket=ozonetest --key=test1/file /tmp/www
   > {
   >     "AcceptRanges": "bytes",
   >     "LastModified": "Wed, 26 Aug 2020 14:24:06 GMT",
   >     "ContentLength": 79,
   >     "ETag": "\"6fcb0dbe8f0ef330b38f81a6a6a7a66e\"",
   >     "ContentType": "text/markdown",
   >     "Metadata": {}
   > }
   > ```
   > 
   > ```
   > aws s3api get-object --bucket=ozonetest --key=test1/file/ /tmp/www
   > 
   > An error occurred (NoSuchKey) when calling the GetObject operation: The specified key does not exist.
   > ```
   > 
   > Just an idea: what do you think about normalizing the path on the client side? (o3fs/ofs). Would it make any problems?
   
   Thank You @elek for the test.
   
   Uploaded design doc has provided reasoning to perform on the OM side.
   
   ![image](https://user-images.githubusercontent.com/8586345/91357044-75c9c900-e7a5-11ea-8eca-b056bf09bb3c.png)
   
   
   And also when used via ofs/o3fs, it uses path Object, and paths are normalized already.
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] elek merged pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
elek merged pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] elek commented on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
elek commented on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-680898429


   Thanks for the patch @bharatviswa504, I would suggest adding a related robot test, too, which would make it possible to test the AWS compatibility of the change.
   
   What do you think?


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] bharatviswa504 edited a comment on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
bharatviswa504 edited a comment on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-684950217


   @elek 
   There is no issue with this, and I believe we have agreed on the part we need this like 100% HCFS with few changes to AWS S3 semantics. I don't see what is the problem in moving forward?. 
   
   As the other argument is when the flag is disabled to support HCFS.
   
   
   Let me know if you have any concerns with this PR.
   
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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


[GitHub] [hadoop-ozone] bharatviswa504 commented on pull request #1328: HDDS-4102. Normalize Keypath for lookupKey.

Posted by GitBox <gi...@apache.org>.
bharatviswa504 commented on pull request #1328:
URL: https://github.com/apache/hadoop-ozone/pull/1328#issuecomment-683905768


   @elek Any more comments?
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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