commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Roger Whitcomb (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (VFS-523) HdfsFileObject equals method returns true for the same path on different hosts
Date Sun, 18 May 2014 05:22:15 GMT

    [ https://issues.apache.org/jira/browse/VFS-523?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14001000#comment-14001000
] 

Roger Whitcomb edited comment on VFS-523 at 5/18/14 5:21 AM:
-------------------------------------------------------------

Hi Bernd,  
The code I'm using is in the Apache Pivot project in the VFSFileBrowser and related classes.
 The failure I noticed, specifically, was that two HdfsFileObjects had the same path on different
file systems, and they had the "equals" method return true.  I believe the code is consistently
calling "equals" for these types of comparisons, but I don't know if all the tests are !equals
or just equals.  I am a little reluctant to say that using just Object.equals would be fine,
because (for instance) I would expect that an existing object pointing to a certain directory
on a given filesystem should be "equal" to a newly created object pointing to that same directory
on the same filesystem, which Object.equals would not return true for (so, I expect this to
work similarly to String.equals where the method returns true if the content is the same,
not merely when the objects are exactly the same object).

So, I will test your latest patch as soon as I return to the office on Tuesday (unfortunately
I am not in a position to try it until then).

Thank you for the investigation on this.
~Roger


was (Author: rwhitcomb):
Hi Bernd,  
The code I'm using is in the Apache Pivot project in the VFSFileBrowser and related classes.
 The failure I noticed, specifically, was that two HdfsFileObjects had the same path on different
file systems, and they had the "equals" method return true.  I believe the code is consistently
calling "equals" for these types of comparisons, but I don't know if all the tests are !equals
or just equals.  I would think that defaulting to the generic Object.equals would probably
be fine.

So, I will test your latest patch as soon as I return to the office on Tuesday (unfortunately
I am not in a position to try it until then).

Thank you for the investigation on this.
~Roger

> HdfsFileObject equals method returns true for the same path on different hosts
> ------------------------------------------------------------------------------
>
>                 Key: VFS-523
>                 URL: https://issues.apache.org/jira/browse/VFS-523
>             Project: Commons VFS
>          Issue Type: Bug
>    Affects Versions: 2.1
>         Environment: Windows 7
>            Reporter: Roger Whitcomb
>            Priority: Minor
>              Labels: hdfs
>             Fix For: 2.1
>
>         Attachments: hdfsequal.diff, unify-hdfs-equals.patch
>
>
> If I construct two HdfsFileObject referring to the same path (for instance, "/") but
on different hosts, the "equals" method will consider them equal (and the "hashCode" method
will give the same results for both) even though they clearly refer to different files since
they are on different hosts.
> These two methods should also factor in the HdfsFileSystem to the equation.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message