search

eclipse / jifa

🔬 Online Heap Dump, GC Log, Thread Dump & JFR File Analyzer.
https://eclipse.github.io/jifa/
Eclipse Public License 2.0
541 stars 96 forks source link

Not able to upload multiple GC log files generated from log rotation #219

Closed altdelnow closed 1 year ago

altdelnow commented 1 year ago

Hi, Currently I am testing jifa in docker mode and found running fine for single log file, irrespective of size. But I am not able to upload multiple files or log.gz file.

Am I missing something ?

parsed_log.zip gc-2023-03-01_14-06-36_21476.log

leveretconey commented 1 year ago

Thanks for your feedback

  1. As to parsed_log.zip, currently jifa does not support compressing multiple rotating logs into a single zip file.

  2. As to gc-2023-03-01_14-06-36_21476.log, I noticed that this log may be using -Xlog:gc=info,heap=trace and this may lead to failure to recognizing format. I will fix this problem and make jifa more rubust.

altdelnow commented 1 year ago

Thanks for response.

Does JIFA supports rotating logs parsing ? As I was testing logs and it failed with message :

[Begin task] Paring 1678087231422-test-gc-mix.log-safe.4 [INFO] Deciding gc log format. org.eclipse.jifa.common.JifaException: org.eclipse.jifa.common.JifaException: Can not recognize format. Is this really a gc log? at org.eclipse.jifa.worker.support.Analyzer.getOrBuild(Analyzer.java:92) at org.eclipse.jifa.worker.support.Analyzer.getOrOpenGCLogModel(Analyzer.java:271) at org.eclipse.jifa.worker.support.Analyzer.analyze(Analyzer.java:140) at org.eclipse.jifa.worker.route.AnalysisRoute.analyze(AnalysisRoute.java:55) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)

leveretconey commented 1 year ago

JIFA does not support rotating logs parsing.

And I have fixed the problem of parsing gc-2023-03-01_14-06-36_21476.log with #220.

altdelnow commented 1 year ago

Thanks a lot. Is this fix available in updated docker image ?

I also realized that JIFA fails to identify individual rotated log. For example if log rotation creates 4 set of logs, JIFA fails of register them as gc logs even individually. Log rotation and zip format of upload will be very important features which JIFA is currently missing. gceasy and GCViewer supports them...

leveretconey commented 1 year ago

The fix has not been available in docker image.

Analyzing individual rotated log should be supported. If JIFA fails to identify individual log, please contact us and we will fix it.