devmadhuu commented on code in PR #6195:
URL: https://github.com/apache/ozone/pull/6195#discussion_r1483918984
##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/KeyManagerImpl.java:
##########
@@ -609,16 +609,20 @@ public ListKeysResult listKeys(String volumeName, String
bucketName,
int maxKeys) throws IOException {
Preconditions.checkNotNull(volumeName);
Preconditions.checkNotNull(bucketName);
-
+ OmBucketInfo omBucketInfo = metadataManager.validateBucket(volumeName,
bucketName);
+ BucketLayout bucketLayout = omBucketInfo.getBucketLayout();
// We don't take a lock in this path, since we walk the
// underlying table using an iterator. That automatically creates a
// snapshot of the data, so we don't need these locks at a higher level
// when we iterate.
-
- if (enableFileSystemPaths) {
+ if (bucketLayout.shouldNormalizePaths(enableFileSystemPaths)) {
startKey = OmUtils.normalizeKey(startKey, true);
keyPrefix = OmUtils.normalizeKey(keyPrefix, true);
}
+ /*startKey = OMClientRequest
Review Comment:
Thanks @tanvipenumudy for review. I have removed the commented code.
--
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]
