Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for , as comma separator in numbers in GC parsing #297

Open
alfh opened this issue Jan 7, 2015 · 0 comments
Open

Add support for , as comma separator in numbers in GC parsing #297

alfh opened this issue Jan 7, 2015 · 0 comments

Comments

@alfh
Copy link
Contributor

alfh commented Jan 7, 2015

In java 8, if I run my JVM with option
"
LC_ALL=nb_NO.UTF-8
export LC_ALL
"
then the gc.log output will actually use the locale based comma separator, which is "," in Norway, and not ".", which is used in US, for example.

So the output will look like this

2015-01-07T08:56:11.600+0100: 2,066: [CMS-concurrent-sweep-start]
2015-01-07T08:56:11.604+0100: 2,070: [CMS-concurrent-sweep: 0,004/0,004 secs] [Times: user=0,03 sys=0,00, real=0,01 secs] 
2015-01-07T08:56:11.604+0100: 2,070: [CMS-concurrent-reset-start]
2015-01-07T08:56:11.612+0100: 2,079: [CMS-concurrent-reset: 0,009/0,009 secs] [Times: user=0,06 sys=0,00, real=0,01 secs] 
2015-01-07T08:56:11.856+0100: 2,323: [GC (GCLocker Initiated GC) 2,323: [ParNew: 157248K->17472K(157248K), 0,0473640 secs] 164764K->66538K(506816K), 0,0474922 secs] [Times: user=0,22 sys=0,02, real=0,05 secs] 
2015-01-07T08:56:12.509+0100: 2,976: [GC (Allocation Failure) 2,976: [ParNew: 157248K->17472K(157248K), 0,0404206 secs] 206314K->98925K(506816K), 0,0405876 secs] [Times: user=0,17 sys=0,00, real=0,04 secs] 
2015-01-07T08:56:12.805+0100: 3,272: [GC (Allocation Failure) 3,272: [ParNew: 157248K->17471K(157248K), 0,0183367 secs] 238701K->111936K(506816K), 0,0184680 secs] [Times: user=0,08 sys=0,00, real=0,02 secs] 
2015-01-07T08:56:13.115+0100: 3,581: [GC (Allocation Failure) 3,581: [ParNew: 157247K->17471K(157248K), 0,0295748 secs] 251713K->128191K(506816K), 0,0297208 secs] [Times: user=0,10 sys=0,00, real=0,03 secs] 
2015-01-07T08:56:13.145+0100: 3,612: [GC (GCLocker Initiated GC) 3,612: [ParNew: 18456K->3411K(157248K), 0,0170746 secs] 129175K->126195K(506816K), 0,0171955 secs] [Times: user=0,06 sys=0,00, real=0,01 secs] 
2015-01-07T08:56:13.763+0100: 4,229: [GC (Allocation Failure) 4,229: [ParNew: 143187K->17472K(157248K), 0,0183644 secs] 265971K->143571K(506816K), 0,0185260 secs] [Times: user=0,08 sys=0,00, real=0,02 secs] 
2015-01-07T08:56:14.588+0100: 5,054: [GC (Allocation Failure) 5,055: [ParNew: 157248K->17471K(157248K), 0,0382582 secs] 283347K->162212K(506816K), 0,0384162 secs] [Times: user=0,20 sys=0,01, real=0,04 secs] 
2015-01-07T08:56:15.850+0100: 6,317: [GC (Allocation Failure) 6,317: [ParNew: 157247K->17471K(157248K), 0,0238880 secs] 301988K->175209K(506816K), 0,0240566 secs] [Times: user=0,12 sys=0,00, real=0,03 secs] 

And this causes the parsing of gc.log to not have any matches.

So I think the regular expressions should support either "," or "." as comma separator in numbers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants