nnguyen168 commented on PR #16595: URL: https://github.com/apache/iceberg/pull/16595#issuecomment-4575763066
I've reopened this PR to get clarification before closing. @rdblue - In my testing, I found that Hadoop's `Path` class normalizes paths and strips trailing slashes. So even if we do: ```java String normalizedDir = dir + "/"; Path path = new Path(normalizedDir); // Path strips the trailing / fs.listStatus(path, pathFilter); ``` The `Path` object will have the trailing slash stripped before it reaches `listStatus()`. **Question**: Is there a different approach you had in mind for ensuring trailing slashes in the Hadoop path? Or is it acceptable that the S3A FileSystem implementation handles this internally when translating directory operations to S3 prefix operations? I want to make sure I understand the expected fix before proceeding or closing 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. 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]
