search

jenkinsci / peass-ci-plugin

Jenkins plugin for peass to support performance measurement in CI
https://plugins.jenkins.io/peass-ci/
GNU Affero General Public License v3.0
4 stars 12 forks source link

Find solution for saving huge Strings while trace-generation #121

Closed mawHBT closed 2 years ago

mawHBT commented 2 years ago

What feature do you want to see added?

de.dagere.peass.dependency.traces.OneTraceGenerator.generateTraceFiles generates Strings which are to long to save:

java.lang.NegativeArraySizeException: -1283060862
    at java.base/java.lang.StringCoding.encodeUTF8(StringCoding.java:904)
    at java.base/java.lang.StringCoding.encode(StringCoding.java:502)
    at java.base/java.lang.String.getBytes(String.java:981)
    at de.dagere.peass.dependency.traces.TraceWriter.writeTraces(TraceWriter.java:54)
    at de.dagere.peass.dependency.traces.TraceWriter.writeTrace(TraceWriter.java:38)
    at de.dagere.peass.dependency.traces.OneTraceGenerator.writeTrace(OneTraceGenerator.java:111)
    at de.dagere.peass.dependency.traces.OneTraceGenerator.generateTraceFiles(OneTraceGenerator.java:104)
    at de.dagere.peass.dependency.traces.OneTraceGenerator.generateTrace(OneTraceGenerator.java:59)

A fix is needed.

Upstream changes

No response

DaGeRe commented 2 years ago

This should work since https://github.com/DaGeRe/peass/commit/f2fcdfb8609efb5f193432f029d438f7cd7c5fae, does this work for you?

mawHBT commented 2 years ago

Is working now.