Looks good! I can sponsor this. Thanks, /Staffan
On 21 jan 2014, at 13:32, Mattias Tobiasson <mattias.tobias...@oracle.com> wrote: > Hi, > Could you please review this test fix. The current test is unstable and fails > intermittently. > > The test parses output from jstat. > It verifies number of header lines, data lines and total lines. > The test sometimes fails because there are unexpected debug output from the > jvm, which means that total line count does not match. > > The fix is to ignore total line count, and only verify header lines and data > lines. > There are multiple tests with the same problem. All have been fixed, even if > the bug only mentions one test. > > bug: > https://bugs.openjdk.java.net/browse/JDK-6545321 > > webrev: > http://cr.openjdk.java.net/~ykantser/6545321/webrev.00/ > > Mattias
