HADOOP-10175. Merging change r1553169 from trunk to branch-2.
git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-2@1553171 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
60f8b513f1
commit
1eaed17cdf
|
@ -273,6 +273,9 @@ Release 2.3.0 - UNRELEASED
|
|||
HADOOP-10087. UserGroupInformation.getGroupNames() fails to return primary
|
||||
group first when JniBasedUnixGroupsMappingWithFallback is used (cmccabe)
|
||||
|
||||
HADOOP-10175. Har files system authority should preserve userinfo.
|
||||
(Chuan Liu via cnauroth)
|
||||
|
||||
Release 2.2.0 - 2013-10-13
|
||||
|
||||
INCOMPATIBLE CHANGES
|
||||
|
|
|
@ -294,6 +294,10 @@ public class HarFileSystem extends FileSystem {
|
|||
private String getHarAuth(URI underLyingUri) {
|
||||
String auth = underLyingUri.getScheme() + "-";
|
||||
if (underLyingUri.getHost() != null) {
|
||||
if (underLyingUri.getUserInfo() != null) {
|
||||
auth += underLyingUri.getUserInfo();
|
||||
auth += "@";
|
||||
}
|
||||
auth += underLyingUri.getHost();
|
||||
if (underLyingUri.getPort() != -1) {
|
||||
auth += ":";
|
||||
|
|
|
@ -258,6 +258,22 @@ public class TestHarFileSystemBasics {
|
|||
0, expectedFileNames.size());
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testMakeQualifiedPath() throws Exception {
|
||||
// Construct a valid har file system path with authority that
|
||||
// contains userinfo and port. The userinfo and port are useless
|
||||
// in local fs uri. They are only used to verify har file system
|
||||
// can correctly preserve the information for the underlying file system.
|
||||
String harPathWithUserinfo = "har://file-user:passwd@localhost:80"
|
||||
+ harPath.toUri().getPath().toString();
|
||||
Path path = new Path(harPathWithUserinfo);
|
||||
Path qualifiedPath = path.getFileSystem(conf).makeQualified(path);
|
||||
assertTrue(String.format(
|
||||
"The qualified path (%s) did not match the expected path (%s).",
|
||||
qualifiedPath.toString(), harPathWithUserinfo),
|
||||
qualifiedPath.toString().equals(harPathWithUserinfo));
|
||||
}
|
||||
|
||||
// ========== Negative:
|
||||
|
||||
@Test
|
||||
|
|
Loading…
Reference in New Issue