[ https://issues.apache.org/jira/browse/HDFS-7025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14124782#comment-14124782 ]
Hadoop QA commented on HDFS-7025: --------------------------------- {color:red}-1 overall{color}. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12667051/HDFS-7025.0.patch against trunk revision 88209ce. {color:green}+1 @author{color}. The patch does not contain any @author tags. {color:green}+1 tests included{color}. The patch appears to include 4 new or modified test files. {color:green}+1 javac{color}. The applied patch does not increase the total number of javac compiler warnings. {color:green}+1 javadoc{color}. There were no new javadoc warning messages. {color:green}+1 eclipse:eclipse{color}. The patch built with eclipse:eclipse. {color:green}+1 findbugs{color}. The patch does not introduce any new Findbugs (version 2.0.3) warnings. {color:green}+1 release audit{color}. The applied patch does not increase the total number of release audit warnings. {color:red}-1 core tests{color}. The patch failed these unit tests in hadoop-hdfs-project/hadoop-hdfs: org.apache.hadoop.hdfs.server.namenode.ha.TestPipelinesFailover {color:green}+1 contrib tests{color}. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/7931//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/7931//console This message is automatically generated. > HDFS Credential Provider related Unit Test Failure > --------------------------------------------------- > > Key: HDFS-7025 > URL: https://issues.apache.org/jira/browse/HDFS-7025 > Project: Hadoop HDFS > Issue Type: Test > Components: encryption > Affects Versions: 2.4.1 > Reporter: Xiaoyu Yao > Assignee: Xiaoyu Yao > Attachments: HDFS-7025.0.patch, HDFS-7025.1.patch > > > Reported by: Xiaomara and investigated by [~cnauroth]. > The credential provider related unit tests failed on Windows. The tests try > to set up a URI by taking the build test directory and concatenating it with > other strings containing the rest of the URI format, i.e.: > {code} > public void testFactory() throws Exception { > Configuration conf = new Configuration(); > conf.set(CredentialProviderFactory.CREDENTIAL_PROVIDER_PATH, > UserProvider.SCHEME_NAME + ":///," + > JavaKeyStoreProvider.SCHEME_NAME + "://file" + tmpDir + > "/test.jks"); > {code} > This logic is incorrect on Windows, because the file path separator will be > '\', which violates URI syntax. Forward slash is not permitted. > The proper fix is to always do path/URI construction through the > org.apache.hadoop.fs.Path class, specifically using the constructors that > take explicit parent and child arguments. > The affected unit tests are: > {code} > * TestCryptoAdminCLI > * TestDFSUtil > * TestEncryptionZones > * TestReservedRawPaths > {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332)