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)