Thank you Stuart http://cr.openjdk.java.net/~tyan/JDK-8030844/webrev.01/ Regards Tristan
On Feb 12, 2014, at 10:06 AM, Stuart Marks <stuart.ma...@oracle.com> wrote: > Hi, yes, I'll take this one. > > It's slightly odd that this is creating filenames that already have "/" in > them (as opposed to File.separator) but since these files don't actually have > to exist, I suppose it doesn't really matter. > > I'm not convinced that we actually have any evidence that /home/~user is > really causing the hang/timeout (either caused by the automounter hanging on > /home or LDAP or other nameservice lookup on ~user), but this is harmless, > and it'll fix the problem on the off chance that this really is the root > cause. > > Tristan, please update the test's @bug tag to add 8030844, create a > changeset, and create a webrev with the changeset in it (as opposed to a bare > patch). I'll then push it for you. > > s'marks > > On 2/10/14 4:08 AM, Alan Bateman wrote: >> On 10/02/2014 10:57, Tristan Yan wrote: >>> Ping: Can anyone give a review on this. >>> Thank you >>> Tristan >> Changing the test so that it doesn't try to /home/~user seems reasonable to >> me. >> >> Stuart - I see you've been sponsoring Tristan's updates to the RMI tests. Are >> you going to take this one too? >> >> -Alan >> >> >>> >>> On Feb 6, 2014, at 5:13 PM, Tristan Yan<tristan....@oracle.com> wrote: >>> >>>> Hi All >>>> >>>> Please help to review a simple fix for >>>> https://bugs.openjdk.java.net/browse/JDK-8030844 >>>> >>>> http://cr.openjdk.java.net/~tyan/JDK-8030844/webrev.00/. >>>> >>>> Description: >>>> Change replace a “/home/~user" folder with an test source path. Folder >>>> “/home/~user” cause some problem whenever something wrong with the >>>> automount >>>> filesystem or an username lookup problem. >>>> >>>> Thank you >>>> Tristan >>