java-sdk icon indicating copy to clipboard operation
java-sdk copied to clipboard

feat: Performance enhancements

Open chrfwow opened this issue 1 month ago • 4 comments

This Pr introduces some performance enhacements, that together yield the following results:

Baseline (AllocationBenchmark.main run five times with the IntelliJ Profiler and averaged): runtime: 5329 ms memory allocations: 16.47 GB

This PR: runtime: 5404ms (no significant change, the deviation of runs is quite large) memory allocations: 14.62 GB

chrfwow avatar Nov 25 '25 12:11 chrfwow

Codecov Report

:x: Patch coverage is 96.87500% with 1 line in your changes missing coverage. Please review. :white_check_mark: Project coverage is 93.66%. Comparing base (ab04409) to head (293c062). :warning: Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...n/java/dev/openfeature/sdk/ImmutableStructure.java 90.00% 0 Missing and 1 partial :warning:
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1741      +/-   ##
============================================
+ Coverage     92.51%   93.66%   +1.15%     
- Complexity      593      601       +8     
============================================
  Files            53       55       +2     
  Lines          1403     1406       +3     
  Branches        150      154       +4     
============================================
+ Hits           1298     1317      +19     
+ Misses           64       50      -14     
+ Partials         41       39       -2     
Flag Coverage Δ
unittests 93.66% <96.87%> (+1.15%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Nov 25 '25 12:11 codecov[bot]

/gemini review

toddbaert avatar Nov 26 '25 17:11 toddbaert

I'll merge this one tomorrow @chrfwow . Up to you if you want to address the minor Gemini issues.

toddbaert avatar Nov 26 '25 17:11 toddbaert