yihua commented on code in PR #11131:
URL: https://github.com/apache/hudi/pull/11131#discussion_r1595870231
##########
hudi-common/src/main/java/org/apache/hudi/common/fs/FSUtils.java:
##########
@@ -635,16 +603,7 @@ public static Long getSizeInMB(long sizeInBytes) {
}
public static Path constructAbsolutePathInHadoopPath(String basePath, String
relativePartitionPath) {
- if (StringUtils.isNullOrEmpty(relativePartitionPath)) {
- return new Path(basePath);
- }
-
- // NOTE: We have to chop leading "/" to make sure Hadoop does not treat it
like
- // absolute path
- String properPartitionPath =
relativePartitionPath.startsWith(PATH_SEPARATOR)
- ? relativePartitionPath.substring(1)
- : relativePartitionPath;
- return constructAbsolutePath(new CachingPath(basePath),
properPartitionPath);
+ return new Path(constructAbsolutePath(basePath,
relativePartitionPath).toUri());
Review Comment:
Got it.
--
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]
