ChenSammi commented on code in PR #8681:
URL: https://github.com/apache/ozone/pull/8681#discussion_r2191759974
##########
hadoop-ozone/common/src/main/java/org/apache/hadoop/ozone/om/helpers/OmLCFilter.java:
##########
@@ -69,8 +82,12 @@ public void valid() throws OMException {
OMException.ResultCodes.INVALID_REQUEST);
}
+ if (hasPrefix && layout == BucketLayout.FILE_SYSTEM_OPTIMIZED) {
+ isValidKeyPath(normalizePrefix(prefix));
Review Comment:
Here is what I'm thinking. First, to distinguish these unsupported cases,
we need to call some function to verify that the prefix is supported or not for
FSO, and throw exception or return error in this case. So either way, we need
to verify the path for FSO. Second, path with "/" or "//" are actually
supported when referring FSO keys(ozone fs -ls), so if we don't support, that
will become one limitation. Your thoughts?
--
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]
