[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17110288#comment-17110288
]
Ayush Saxena commented on HDFS-14762:
-
Seems this a very old and known one, check HDFS-13
Checking
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17110069#comment-17110069
]
Surendra Singh Lilhore commented on HDFS-14762:
---
Hi [~ayushtkn],
{quote}File Name used
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17109947#comment-17109947
]
hemanthboyina commented on HDFS-14762:
--
i think this issue also relates to IPV6
>
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17109943#comment-17109943
]
Ayush Saxena commented on HDFS-14762:
-
bq. some file name used IPv6.
File Name used IPv6? What is the
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16952404#comment-16952404
]
Shixiong Zhu commented on HDFS-14762:
-
[~ayushtkn] Some file systems allow ":" in the name. For
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=1693#comment-1693
]
Ayush Saxena commented on HDFS-14762:
-
Sorry for the goof up, but just realized, name having : isn't
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16929777#comment-16929777
]
hemanthboyina commented on HDFS-14762:
--
thanks for the new test case condition [~ayushtkn]
Updated
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16929764#comment-16929764
]
Hadoop QA commented on HDFS-14762:
--
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16929678#comment-16929678
]
Ayush Saxena commented on HDFS-14762:
-
If the path just has colon at end? say a: ?
>
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16929645#comment-16929645
]
hemanthboyina commented on HDFS-14762:
--
no [~ayushtkn] that doesn't work
we need to update the
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16929575#comment-16929575
]
Ayush Saxena commented on HDFS-14762:
-
Will just having this work :
{code:java}
// add "./" in
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16921182#comment-16921182
]
hemanthboyina commented on HDFS-14762:
--
updated the patch , no test failures now
>
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16921179#comment-16921179
]
Hadoop QA commented on HDFS-14762:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16920552#comment-16920552
]
Hadoop QA commented on HDFS-14762:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16920452#comment-16920452
]
Ayush Saxena commented on HDFS-14762:
-
Test failures seems related. Please check!!!
>
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16920435#comment-16920435
]
hemanthboyina commented on HDFS-14762:
--
Updated the patch , pls review [~ayushtkn]
>
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16920336#comment-16920336
]
Hadoop QA commented on HDFS-14762:
--
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16920303#comment-16920303
]
hemanthboyina commented on HDFS-14762:
--
{code:java}
// add "./" in front of Linux relative paths
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16919769#comment-16919769
]
Shixiong Zhu commented on HDFS-14762:
-
I don't have a proposal right now. Fixing the Path constructor
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16919244#comment-16919244
]
hemanthboyina commented on HDFS-14762:
--
hi [~ayushtkn] [~zsxwing] written a unit test with "a:b" ,
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16919207#comment-16919207
]
Ayush Saxena commented on HDFS-14762:
-
[~zsxwing] do you propose any other way of fixing this,
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16918897#comment-16918897
]
hemanthboyina commented on HDFS-14762:
--
fixing the issue at constructor level will make sure Path
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16918865#comment-16918865
]
Ayush Saxena commented on HDFS-14762:
-
IMO we should try fixing {{"Path(Path/String parent, String
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16918827#comment-16918827
]
hemanthboyina commented on HDFS-14762:
--
[~zsxwing] root cause fix should be at Path constructor , i
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16916118#comment-16916118
]
Shixiong Zhu commented on HDFS-14762:
-
[~hemanthboyina] I'm not concerned about the behavior of this
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16915242#comment-16915242
]
hemanthboyina commented on HDFS-14762:
--
[~zsxwing] , then we need to handle the exception with a
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16915241#comment-16915241
]
hemanthboyina commented on HDFS-14762:
--
[~zsxwing] , then we need to handle the exception with a
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16914685#comment-16914685
]
Shixiong Zhu commented on HDFS-14762:
-
IMO, "a:b" is ambiguous. It can be a relative path or a path
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16914575#comment-16914575
]
hemanthboyina commented on HDFS-14762:
--
{code:java}
// parse uri scheme, if any
int colon =
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16913583#comment-16913583
]
Ayush Saxena commented on HDFS-14762:
-
Thanx [~zsxwing] and [~hemanthboyina]. Ideally it shoudln't
[
https://issues.apache.org/jira/browse/HDFS-14762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16913340#comment-16913340
]
hemanthboyina commented on HDFS-14762:
--
written a junit , the exception was thrown if the child
31 matches
Mail list logo
