Re: RFR: 8027822: ProblemList.txt Updates (11/2013)

2013-11-07 Thread Chris Hegarty
Sorry for the delay Amy. Trivially, 'windows-amd64, solaris-amd64, linux-amd64, macosx-all' was not matching the 64bit versions of these OSes, neither was s/amd64/x64/. So I changed it to generic-all for now. I did run into this myself a while back, but can't remember the correct labels. Not

Re: RFR: 8027822: ProblemList.txt Updates (11/2013)

2013-11-07 Thread Amy Lu
Thank you Chris ! /Amy On 11/7/13 5:09 PM, Chris Hegarty wrote: Sorry for the delay Amy. Trivially, 'windows-amd64, solaris-amd64, linux-amd64, macosx-all' was not matching the 64bit versions of these OSes, neither was s/amd64/x64/. So I changed it to generic-all for now. I did run into

Re: RFR: 8027822: ProblemList.txt Updates (11/2013)

2013-11-07 Thread Aleksej Efimov
Hi Amy, This test is currently fixed and working as expected, but I have a desire to remove this test. In fact, this test (and my decision to fix and exclude it) already brings too many question, it doesn't do any helpful testing and all situation with adding fixed test to ProblemsList.txt

Re: RFR: 8027822: ProblemList.txt Updates (11/2013)

2013-11-07 Thread Amy Lu
Thank you for confirm. And yes, removed. Thanks, Amy On 11/7/13 6:36 PM, Aleksej Efimov wrote: Hi Amy, This test is currently fixed and working as expected, but I have a desire to remove this test. In fact, this test (and my decision to fix and exclude it) already brings too many question,

Re: RFR: 8027822: ProblemList.txt Updates (11/2013)

2013-11-06 Thread Amy Lu
On 11/6/13 10:45 AM, Amy Lu wrote: On 11/5/13 7:09 PM, Amy Lu wrote: On 11/5/13 6:01 PM, Alan Bateman wrote: On 05/11/2013 09:26, Amy Lu wrote: : Please review ProblemList.txt changes: https://dl.dropboxusercontent.com/u/5812451/yl153753/8027822/webrev/index.html This looks okay to me

RFR: 8027822: ProblemList.txt Updates (11/2013)

2013-11-05 Thread Amy Lu
Following bugs have been fixed, related tests should be removed from ProblemList: 8021230 (CODETOOLS-7900208 https://bugs.openjdk.java.net/browse/CODETOOLS-7900208has been promoted in jtreg 4.1 b07) java/lang/ThreadLocal/ThreadLocalSupplierTest.java generic-all JDK-8026502

Re: RFR: 8027822: ProblemList.txt Updates (11/2013)

2013-11-05 Thread Chris Hegarty
Thanks Amy, The updates look good to me. Just to confirm, have you verified that the tests actually pass now? -Chris. On 05/11/2013 09:26, Amy Lu wrote: Following bugs have been fixed, related tests should be removed from ProblemList: 8021230 (CODETOOLS-7900208

Re: RFR: 8027822: ProblemList.txt Updates (11/2013)

2013-11-05 Thread Alan Bateman
On 05/11/2013 09:26, Amy Lu wrote: : Please review ProblemList.txt changes: https://dl.dropboxusercontent.com/u/5812451/yl153753/8027822/webrev/index.html This looks okay to me although I'm not sure about sun/util/resources/TimeZone/Bug6317929.java (for some reason Aleksej fixed the test

Re: RFR: 8027822: ProblemList.txt Updates (11/2013)

2013-11-05 Thread Amy Lu
On 11/5/13 6:01 PM, Alan Bateman wrote: On 05/11/2013 09:26, Amy Lu wrote: : Please review ProblemList.txt changes: https://dl.dropboxusercontent.com/u/5812451/yl153753/8027822/webrev/index.html This looks okay to me although I'm not sure about sun/util/resources/TimeZone/Bug6317929.java

Re: RFR: 8027822: ProblemList.txt Updates (11/2013)

2013-11-05 Thread Amy Lu
On 11/5/13 6:01 PM, Chris Hegarty wrote: Thanks Amy, The updates look good to me. Just to confirm, have you verified that the tests actually pass now? Yes, I tried them on one linux (as they are generic-all or linux-all) and test PASS. Thanks, Amy -Chris. On 05/11/2013 09:26, Amy Lu

Re: RFR: 8027822: ProblemList.txt Updates (11/2013)

2013-11-05 Thread Chris Hegarty
Thanks Amy, I can sponsor this change for you. -Chris. On 05/11/2013 11:09, Amy Lu wrote: On 11/5/13 6:01 PM, Alan Bateman wrote: On 05/11/2013 09:26, Amy Lu wrote: : Please review ProblemList.txt changes: https://dl.dropboxusercontent.com/u/5812451/yl153753/8027822/webrev/index.html

Re: RFR: 8027822: ProblemList.txt Updates (11/2013)

2013-11-05 Thread Amy Lu
On 11/5/13 7:09 PM, Amy Lu wrote: On 11/5/13 6:01 PM, Alan Bateman wrote: On 05/11/2013 09:26, Amy Lu wrote: : Please review ProblemList.txt changes: https://dl.dropboxusercontent.com/u/5812451/yl153753/8027822/webrev/index.html This looks okay to me although I'm not sure about

Re: RFR: 8027822: ProblemList.txt Updates (11/2013)

2013-11-05 Thread Tristan Yan
Hi Amy Can we exclude below two also test/com/sun/net/httpserver/Test9a.java test/java/util/concurrent/ThreadPoolExecutor/CoreThreadTimeOut.java Thank you Tristan On 06/11/2013 10:45, Amy Lu wrote: On 11/5/13 7:09 PM, Amy Lu wrote: On 11/5/13 6:01 PM, Alan Bateman wrote: On 05/11/2013

Re: RFR: 8027822: ProblemList.txt Updates (11/2013)

2013-11-05 Thread Tristan Yan
Sorry, I replied to the wrong thread, please ignore my previous mail Tristan On 06/11/2013 11:04, Tristan Yan wrote: Hi Amy Can we exclude below two also test/com/sun/net/httpserver/Test9a.java test/java/util/concurrent/ThreadPoolExecutor/CoreThreadTimeOut.java Thank you Tristan On