RFR: 8057747: Several test failing after update to tzdata2014g
Hello, There was a problem during tzdata2014g update with JSR310 implementation. It causes reported test failure: jdk/test/sun/util/calendar/zi/TestZoneInfo310.java. The https://bugs.openjdk.java.net/browse/JDK-8051641 bug was filled for this failure, but the test wasn't added to the ProblemList.txt. Please, review the addition of this test to ProblemList.txt: http://cr.openjdk.java.net/~aefimov/8057747/webrev.00 -Aleksej
On 08/09/2014 10:26, Aleksej Efimov wrote:
Hello,
There was a problem during tzdata2014g update with JSR310 implementation. It causes reported test failure: jdk/test/sun/util/calendar/zi/TestZoneInfo310.java. The https://bugs.openjdk.java.net/browse/JDK-8051641 bug was filled for this failure, but the test wasn't added to the ProblemList.txt. Please, review the addition of this test to ProblemList.txt: http://cr.openjdk.java.net/~aefimov/8057747/webrev.00 Looks okay (although "Filed" is not needed in the comment). Hopefully the issue with the JSR 310 test can be tracked down quickly. If you do push this change then it would be useful to add a note to JDK-8051641 so that whoever takes it remembers to remove it from the exclude list.
-Alan
Thank you for review Alan! Will remove the "Filed" before the push and will add a note. -Aleksej On 09/08/2014 02:38 PM, Alan Bateman wrote:
On 08/09/2014 10:26, Aleksej Efimov wrote:
Hello,
There was a problem during tzdata2014g update with JSR310 implementation. It causes reported test failure: jdk/test/sun/util/calendar/zi/TestZoneInfo310.java. The https://bugs.openjdk.java.net/browse/JDK-8051641 bug was filled for this failure, but the test wasn't added to the ProblemList.txt. Please, review the addition of this test to ProblemList.txt: http://cr.openjdk.java.net/~aefimov/8057747/webrev.00 Looks okay (although "Filed" is not needed in the comment). Hopefully the issue with the JSR 310 test can be tracked down quickly. If you do push this change then it would be useful to add a note to JDK-8051641 so that whoever takes it remembers to remove it from the exclude list.
-Alan
participants (2)
-
Alan Bateman
-
Aleksej Efimov