Can I get one more review please?
thanks,
Chris
On 12/3/19 1:10 PM, serguei.spit...@oracle.com wrote:
Hi Chris,
It looks good.
Thanks,
Serguei
On 12/3/19 12:45 PM, Chris Plummer wrote:
Hello,
Please review the following:
https://bugs.openjdk.java.net/browse/JDK-8234277
http://cr.openjdk.java.net/~cjplummer/8234277/webrev.00/
No longer redirect stderr for the jhsdb/clhsdb process. It results in
not seeing attach failures in the output, so OutputAnalyer can't
check for them.
Execute "verbose true" as the first clhsdb command after launching.
This will result in verboseExceptions being true in
CommandProcessor.java, so full exception traces will appear in the
output. This will make debugging future SA test failures a lot easier.
Add an extra check for any DebuggerException. This is mainly for
detecting that the attached failed. This previously was going
un-noticed, and instead the test would later fail because it noticed
some other issue, like missing output, which isn't very informative.
Add checks for other unexpected SA exceptions that are caught and
printed by CommandProcessor. These will always have an "Error: "
prefix, making them easy to detect.
Problem list ClhsdbScanOops.java. With the new error checking, it
will now always fail on windows due to JDK-8230731 and on macos and
linux due to JDK-8235220. These failures are not "new" per se, but
are just now being properly detected.
thanks,
Chris