RFR: 8291081: Some sun/tools/jstatd/TestJstatd* tests fail with "Not a percentage\: 68.31\: expected true, was false" [v3]

[Kevin Walls]

On Tue, 9 Aug 2022 20:29:42 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

>> The test should use the same locales in all processes, the default language should work fine.
>
> Leonid Mesnik has updated the pull request incrementally with one additional commit since the last revision:
> 
>   fixed print

Marked as reviewed by kevinw (Committer).

OK, I did not know that user.language=vi was the trick to make it fail.  With that I see the failure (at last! 8-) ).
(I had tried other European languages that use , as a decimal separator, and don't fail.  Different thousands separator behaviour or just values that weren't out of range?...)

You get 91.10 parsed as 9110.0 which seems odd, it's not 91 thousand... But I find comments that grouping (thousands) chars might be ignored when parsing.  So mystery is I think solved, and the test is improved! 8-)

-------------

PR: https://git.openjdk.org/jdk/pull/9798