-
Notifications
You must be signed in to change notification settings - Fork 978
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Support for ShenandoahGC in non unified jvm logging format #198
Comments
Hi Jeroen
Did you check the latest SNAPSHOT version
(https://github.com/chewiebug/GCViewer/wiki/Changelog)? Is it missing
something concerning Shenandoah support?
Best regards,
Jörg
|
Hi Jorg, |
Hi Jorg, It fails to parse the gc logfile, I get: INFO [DataReaderFacade]: GCViewer version 1.36-SNAPSHOT (2018-05-30T19:10:38+0000) These are the first 3 lines in de log: Regards, |
Hi Jeroen,
Ah, now I see! You are using Shenandoah with Java 8, which uses the
"old" gc format. GCViewers implementation is only able to parse
Shenandoah with the Unified Logging Format available starting with JDK
9. Out of curiosity: Is there a particular reason, why you are using
Shenandoah with Java 8?
If you attach a full sample file to this issue, I might be able to add
support for the "old" format, as well.
Best regards,
Jörg
|
Hi Jorg, Right :-) Regards, |
Hi Jeroen
Hmm, I see, that my last answer didn't make it back to github...
The files are nice! I have pushed a commit, which is able to parse both
files (you may download
<https://github.com/chewiebug/GCViewer/wiki/Changelog> the latest
SNAPSHOT build).
I see, that you didn't use -XX:+PrintGCDetails. Usually, the addition of
that option adds more information for gc analysis (and also needs more
tweaking of the parser). Are you able to provide a sample file with this
option turned on? Or was there a special reason, why you didn't use that
option?
Best regards,
Jörg
|
Hi Jörg, Ah, great, will try. Regards, |
Hi Jörg, See attachment. I cannot get the tool analyze the file. I get: Regards, |
Hi Jeroen,
Thank you for the sample file!
If you use the latest 1.36-SNAPSHOT version of GCViewer
(2018-08-01T15:37:51+0000), the parser will recognise the file, but show
a lot of warnings. I'll probably release a fix for those warnings, soon.
Regards,
Jörg
|
Hi Jeroen, I have pushed the fix. The latest SNAPSHOT version is able to parse your sample file. Regards, |
Hi Jörg, Great! it works. Very nice. What I however not understand is why the results are different in GCEasy. There the max gc pause is 0,029 s. while in GCViewer it is 0,020 s. I will attach two more Shenandoah gc log files for which GCViewer shows several Warnings; it would be nice if those get fixed as well. Regards, |
Great to hear, that it works!
I have done some comparing numbers with the results gceasy.io provides. I don't really understand the differences. Comparing numbers with "Shenandoah Collection Phases Statistics" in gceasy and "Event details" in GCViewer, I see no difference. "Shenandoah GC Time" -> "Pause Time" shows different numbers. I'd expect them to match the "Shenandoah Collection Phases Statistics" and GCViewers "Pause" -> "Total Pause" numbers, but they dont' (apart from the total pause time of 12.54s).
I'll have a look. Regards, |
Hi, In the lastest version (2018-10-23) I don't get to see anything useful when loading our Shenandoah gc log file. Only ParseExceptions. I attached a log file. |
I have improved the parser. I think, it won't parse the latest sample you attached, because Shenandoah changed it's output significantly. Please open another issue, if you find issues with Shenandoah, again. |
By the way: There is a new release 1.36 |
Thanks!
Kind regards, met vriendelijke groet,
Jeroen Borgers
drs. Jeroen Borgers
Principal Consultant
jPinpoint Performance Services, jpinpoint com, The Netherlands.
Mobile: +31(0)6-30128951, e-mail: jborgers at jpinpoint com
Op za 30 nov. 2019 om 23:00 schreef chewiebug <notifications@github.com>:
… By the way: There is a new release 1.36
<https://github.com/chewiebug/GCViewer/releases/tag/1.36>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#198?email_source=notifications&email_token=AF3TVW2277KBEWDTOIEVEFTQWLO3HA5CNFSM4FD7F6QKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFQUYRY#issuecomment-560024647>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AF3TVW2F7UQA7IAUUQZXYVTQWLO3HANCNFSM4FD7F6QA>
.
|
…vm logging format
…rmat' into develop
Hi,
As a GC tuner I'd like GCViewer to support ShenandoahGC, the ultra low pause time collector.
See https://wiki.openjdk.java.net/display/shenandoah/Main
Best, Jeroen
The text was updated successfully, but these errors were encountered: