[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