[
https://issues.apache.org/jira/browse/HADOOP-10889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14072151#comment-14072151
]
Yongjun Zhang commented on HADOOP-10889:
----------------------------------------
Thanks Arpit! I totally agree. Let's try to fix all in this jira. Like you
said, we need to find testcases that don't set the test root of their own. I
actually did some search earlier, saw "target/test/data", "build/test/data" and
"/tmp" are used at different places. We can standardize on it too. I prefer
"build/test/data".
BTW, thanks for changing the title of this jira, it's certainly much better one.
> Verify usage of test.build.data in test cases
> ---------------------------------------------
>
> Key: HADOOP-10889
> URL: https://issues.apache.org/jira/browse/HADOOP-10889
> Project: Hadoop Common
> Issue Type: Bug
> Reporter: Yongjun Zhang
> Assignee: Yongjun Zhang
>
> Per [~arpitagarwal]'s comments in HDFS-6719, I'm filing this jira as a
> follow-up. The goal is to fix the misuse of test.build.data over quite some
> places. Thanks Arpit!
> {code}
> FSTestWrapper.java
> FileContextMainOperationsBaseTest.java
> FileContextTestHelper.java
> FileContextURIBase.java
> FileSystemTestHelper.java
> MiniDFSCluster.java
> TestBlocksWithNotEnoughRacks.java
> TestChecksumFileSystem.java
> TestCopyPreserveFlag.java
> TestCreateEditsLog.java
> TestDFSUpgradeFromImage.java
> TestDecommissioningStatus.java
> TestEnhancedByteBufferAccess.java
> TestFSImageWithSnapshot.java
> TestFileUtil.java
> TestFsShellReturnCode.java
> TestHadoopArchives.java
> TestHarFileSystemBasics.java
> TestHardLink.java
> TestHdfsTextCommand.java
> TestHostsFiles.java
> TestJHLA.java
> TestListFiles.java
> TestLocalFileSystem.java
> TestNameNodeRecovery.java
> TestNativeIO.java
> TestPathData.java
> TestPread.java
> TestRenameWithSnapshots.java
> TestSeekBug.java
> TestSlive.java
> TestSnapshot.java
> TestStartup.java
> TestTextCommand.java
> etc
> {code}
--
This message was sent by Atlassian JIRA
(v6.2#6252)
