RFR (XS) 8202360: [TESTBUG] runtime/LoadClass/TestResize.java needs to print output when it fails

[David Holmes]

Hi Gerard,

On 18/05/2018 2:21 AM, Gerard Ziemski wrote:
> hi David,
> 
> Thank you for the review!
> 
> 
>> On May 16, 2018, at 4:54 PM, David Holmes <david.holmes at oracle.com> wrote:
>>
>> Hi Gerard,
>>
>> On 17/05/2018 3:47 AM, Gerard Ziemski wrote:
>>> Hi all,
>>> Please review this small enhancement where we print out additional information (i.e. output from PrintSystemDictionaryAtExit), when we detect the failure.
>>> https://bugs.openjdk.java.net/browse/JDK-8202360
>>> http://cr.openjdk.java.net/~gziemski/8202360_rev1/
>>
>> I don't understand what your fix is doing - sorry.
> 
> The fix takes all the available output bytes from the process (using Scanner to grab all tokens at once) and prints all the output from the process (i.e. PrintSystemDictionaryAtExit, and there is a lot of it) at failure.

Okay but that's the rest of the output that hasn't yet been processed - 
correct? So the line of output that caused the failure won't be there as 
it's already been grabbed by the scanner and processed.

>>
>> What I wanted to see was the complete history of lines of the form
>>
>> "Java dictionary (table_size=107, classes=6)"
>>
>> so that we can see how the table size and number of classes have changed leading up to the failure. But I realize now that we already get that from the:
>>
>> System.out.println("PASS table_size:"+table_size+", classes:"+classes+" OK");
>>
>> All that is missing is the printing of the table_size and classes when we throw the exception. That can be trivially fixed:
>>
>>          if (loadFactor > MAX_LOAD_FACTOR) {
>> +          System.out.println("FAIL table_size:"+table_size+", classes:"+classes);
>>            throw new RuntimeException("Load factor too high, expected MAX "+MAX_LOAD_FACTOR+", got "+loadFactor);
>>          } else {
>>            System.out.println("PASS table_size:"+table_size+", classes:"+classes+" OK");
>>          }
> 
> OK, so you determined that only that one piece of info was missing in the end in your case, but maybe the next person might want to see everything that PrintSystemDictionaryAtExit provides?
> 
> As long as we’re touching the test - should we then stick with the solution that prints out everything from PrintSystemDictionaryAtExit?

I'm not sure how anything that comes later in 
PrintSystemDictionaryAtExit would help with a failure that happened 
earlier, but if you see value it then that's fine. However I think you 
need my suggestion as well to get the information that actually caused 
the failure.

Some sample output in the bug report would be useful.

Thanks,
David

> 
> cheers
> 
>