Hi Arno, thanks for the 'r'eview!
Best regards, Goetz. > -----Original Message----- > From: Zeller, Arno > Sent: Mittwoch, 20. Juni 2018 10:12 > To: Lindenmaier, Goetz <[email protected]> > Cc: serviceability-dev ([email protected]) <serviceability- > [email protected]>; Thomas Stüfe <[email protected]> > Subject: RE: RFR(XS): 8205108: [testbug] Fix pattern matching in jstatd tests. > > Hi Goetz, > > the change looks good to me and I think the newer example output should > be enough. > Nevertheless, I have to point out that I am no reviewer. > > Best regards, > Arno > > >-----Original Message----- > >From: serviceability-dev [mailto:serviceability-dev- > >[email protected]] On Behalf Of Lindenmaier, Goetz > >Sent: Dienstag, 19. Juni 2018 12:46 > >To: Thomas Stüfe <[email protected]> > >Cc: serviceability-dev ([email protected]) <serviceability- > >[email protected]> > >Subject: [CAUTION] RE: RFR(XS): 8205108: [testbug] Fix pattern matching in > >jstatd tests. > > > >Hi Thomas, > > > >thanks for your review! > > > >I think the upper example output stems from some older > >jstatd. Probably the output got two new values added. > >I don't feel like making up values, so I'd rather remove the > >old output. Is that fine with you? > > > >Best regards, > > Goetz. > > > >> -----Original Message----- > >> From: Thomas Stüfe [mailto:[email protected]] > >> Sent: Samstag, 16. Juni 2018 19:26 > >> To: Lindenmaier, Goetz <[email protected]> > >> Cc: serviceability-dev ([email protected]) > <serviceability- > >> [email protected]> > >> Subject: Re: RFR(XS): 8205108: [testbug] Fix pattern matching in jstatd > tests. > >> > >> Hi Goetz, > >> > >> Looks reasonable. Could you please fix up the example output above the > >> alternative you added, it misses the columns "CGC" and "CGCT". > >> > >> Thanks, Thomas > >> > >> On Sat, Jun 16, 2018 at 9:24 AM, Lindenmaier, Goetz > >> <[email protected]> wrote: > >> > Hi, > >> > > >> > Please review this test fix: > >> > http://cr.openjdk.java.net/~goetz/wr18/8205108- > >fixJstatDTestPattern/01/ > >> > > >> > The values for 'M' must be parsed as PERCENTAGE_OR_DASH. > >> > The parsing of numbers should not depend on the locale. > >> > I changed the parsing to use java.text.NumberFormat > >> > > >> > Best regards, > >> > Goetz. > >> >
