[
https://issues.apache.org/jira/browse/HDFS-11434?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15877474#comment-15877474
]
Vinayakumar B commented on HDFS-11434:
--------------------------------------
Directly changing the {{PathUtils}} may add confusion where this method is
called to get the local filesystem path. Ex: {{TestDiskBalancerCommand}}.
TestQuota (and possibly other) usages of PathUtils are to get the randomized
HDFS path, not Local Path. Since each running test have its own HDFS cluster to
run with, randomness in path not required. It can run with simple path names
such as '/testquota/test1'. So I feel for the current issue fix, patch v1 looks
better.
> TestQuota fails in windows
> --------------------------
>
> Key: HDFS-11434
> URL: https://issues.apache.org/jira/browse/HDFS-11434
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: test
> Reporter: Brahma Reddy Battula
> Assignee: Brahma Reddy Battula
> Attachments: HDFS-11434-002.patch, HDFS-11434.patch
>
>
> {noformat}
> java.lang.IllegalArgumentException: Pathname
> /D:/hadoop-trunk/hadoop/hadoop-hdfs-project/hadoop-hdfs/target/test/data/qTr6mgA9Yv/TestQuota/testQuotaCommands
> from
> D:/hadoop-trunk/hadoop/hadoop-hdfs-project/hadoop-hdfs/target/test/data/qTr6mgA9Yv/TestQuota/testQuotaCommands
> is not a valid DFS filename.
> at
> org.apache.hadoop.hdfs.DistributedFileSystem.getPathName(DistributedFileSystem.java:212)
> at
> org.apache.hadoop.hdfs.DistributedFileSystem$25.doCall(DistributedFileSystem.java:1119)
> at
> org.apache.hadoop.hdfs.DistributedFileSystem$25.doCall(DistributedFileSystem.java:1116)
> at
> org.apache.hadoop.fs.FileSystemLinkResolver.resolve(FileSystemLinkResolver.java:81)
> at
> org.apache.hadoop.hdfs.DistributedFileSystem.mkdirsInternal(DistributedFileSystem.java:1133)
> at
> org.apache.hadoop.hdfs.DistributedFileSystem.mkdirs(DistributedFileSystem.java:1108)
> at org.apache.hadoop.fs.FileSystem.mkdirs(FileSystem.java:2218)
> {noformat}
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
