[crac] RFR: Fixing VMOptionsTest.java
    Roman Marchenko 
    rmarchenko at openjdk.org
       
    Wed Jun 14 12:21:22 UTC 2023
    
    
  
On Tue, 13 Jun 2023 14:59:46 GMT, Radim Vansa <duke at openjdk.org> wrote:
>> @rvansa Could you review please?
>
> Hi @wkia , the PR looks correct, but I wonder if we need to complicate things if the NMT default is not the same in different builds. I've picked this simply as a random specimen of the non-manageable opts (with string args); could you just use something else, e.g. `OnOutOfMemoryError`, or if you want to go with non-empty default there's `MetaspaceReclaimPolicy`?
@rvansa 
> did you find any issues with using the same bean after restore?
No, there is no problem with it, I just tried to make the checks isolated to prevent any interference.
-------------
PR Comment: https://git.openjdk.org/crac/pull/81#issuecomment-1591083283
    
    
More information about the crac-dev
mailing list