Looks good to me (not a Reviewer however). Best regards, Matthias
> > Message: 2 > Date: Mon, 18 Feb 2019 15:35:20 +0000 > From: "Langer, Christoph" <christoph.lan...@sap.com> > To: "Zeller, Arno" <arno.zel...@sap.com>, Nishit Jain > <nishit.j...@oracle.com>, Naoto Sato <naoto.s...@oracle.com>, > Roger > Riggs <roger.ri...@oracle.com> > Cc: core-libs-dev <core-libs-dev@openjdk.java.net> > Subject: RE: RFR(XS):JDK-8219228: > java/util/Base64/TestEncodingDecodingLength.java failing on 8GB > test > machine. > Message-ID: <a01efd886d8844b5939b09a34163d...@sap.com> > Content-Type: text/plain; charset="us-ascii" > > Hi Arno, > > this looks good and trivial. I'll sponsor this one. > > Maybe one of the authors/reviewers of the original fix for bug > https://bugs.openjdk.java.net/browse/JDK-8218265 (Nishit, Roger, Naoto) > should approve this. > > Thanks > Christoph > > > > -----Original Message----- > > From: core-libs-dev <core-libs-dev-boun...@openjdk.java.net> On Behalf > > Of Zeller, Arno > > Sent: Montag, 18. Februar 2019 13:49 > > To: core-libs-dev <core-libs-dev@openjdk.java.net> > > Subject: [CAUTION] RFR(XS):JDK-8219228: > > java/util/Base64/TestEncodingDecodingLength.java failing on 8GB test > > machine. > > > > Hello! > > > > I found that the test java/util/Base64/TestEncodingDecodingLength.java > fails > > on a machine with 8GB memory after JDK-8218265. > > The test starts a VM with -Xmx8GB but the VM needs some more memory > > than only the heap and on machines with just 8GB of memory (and no > swap > > configured) the test will always fail because the VM cannot get enough > > native memory. > > Therefore I suggest to increase "@requires os.maxMemory" to >= 10GB to > > be safe. > > > > Could someone please review this minimal change? > > > > Bug: https://bugs.openjdk.java.net/browse/JDK-8219228 > > > > Webrev: http://cr.openjdk.java.net/~azeller/webrevs/8219228/ > > > > Thanks and best regards, > > Arno