Hi Sharath,

Your changes look good. One question:

Was the following test excluded by mistake ?

serviceability/sa/jmap-hprof/JMapHProfLargeHeapTest.java

Would you please also delete these 2 lines since these tests have been removed ?

serviceability/sa/ClhsdbSymbol.java 8193639 solaris-all
serviceability/sa/ClhsdbSymbolTable.java 8193639 solaris-all

Thanks,
Jini


On 8/14/2018 12:48 PM, Sharath Ballal wrote:
Hello,

Pls review this small fix to problem list SA testcases failing on Solaris.

Bug ID: https://bugs.openjdk.java.net/browse/JDK-8209342

Webrev: http://cr.openjdk.java.net/~sballal/8209342/webrev.00/ <http://cr.openjdk.java.net/%7Esballal/8209342/webrev.00/>

Thanks,

Sharath

Reply via email to