You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Chesnay Schepler (JIRA)" <ji...@apache.org> on 2018/06/05 08:50:00 UTC

[jira] [Created] (FLINK-9527) Paths initialize with URI are not normalized

Chesnay Schepler created FLINK-9527:
---------------------------------------

             Summary: Paths initialize with URI are not normalized
                 Key: FLINK-9527
                 URL: https://issues.apache.org/jira/browse/FLINK-9527
             Project: Flink
          Issue Type: Bug
          Components: Core
    Affects Versions: 1.4.2, 1.5.0, 1.3.3, 1.2.1, 1.1.4, 1.0.3, 1.6.0
            Reporter: Chesnay Schepler


The {{Path}} class has a lot of logic to normalize paths, like replacing {{\\}} with {{/}} and removing trailing slashes.

However, all this normalization can be circumvented by passing in a {{URI}}, where no normalization is done. I guess this was done on the assumption that the normalization was compatible with {{URI}}, but it turns out that this is not the case in the case of trailing slashes.

An {{URI}} may have a trailing slash for directories, whereas a normalized {{Path}} does not.

Unfortunately this breaks some assumptions made in the {{Path}} class, in particular {{Path#getParent()}} does not properly handle this case.

Both assertions in the code below will fail.
{code}
public void testPathNormalization() {
		String path = "C:/Users/Zento/AppData/Local/Temp/test/";
		URI uri = URI.create(path);

		Path normalizedPath = new Path(path);
		Path uriPath = new Path(uri);

		Assert.assertEquals(normalizedPath.toString(), uriPath.toString());
		Assert.assertEquals("C:/Users/Zento/AppData/Local/Temp", uriPath.getParent().getPath());
	}
{code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)