Skip to content

docs: tuning guidance from the 1.0 load tests - #236

Merged
wolf4ood merged 1 commit into
eclipse-edc:mainfrom
wolf4ood:docs/tuning_bench_findings
Oct 30, 2026
Merged

wolf4ood merged 1 commit into
eclipse-edc:mainfrom
wolf4ood:docs/tuning_bench_findings

Conversation

@wolf4ood

Copy link
Copy Markdown
Contributor

What this PR changes/adds

Extends the Tuning page with what the 1.0 load tests showed (multi-tenant runtime, up to 100 participants,
negotiation + transfer at 40+ flows/s):

  • Database connections: corrects the catalog statement after fix: verify the DSP catalog request token outside the transaction Connector#6180 (the token is now validated
    before the transaction), and adds the NATS workers to the pool sizing.
  • DID resolution (new): edc.did.resolver.cache.size / .expiry. The cache is shared by all hosted participants,
    and a miss is a synchronous HTTP fetch on the request thread. With 99 counter-parties the default of 50 entries
    raised the median negotiation time from ~1 s to 11 s.
  • Finding the limit: two more symptoms, all NATS workers busy and threads in WebDidResolver.resolve.
  • Policy evaluation (new): edc.policy.cel.expressions.cache.ttl.ms, and what it means for multi-replica setups.
  • JVM memory (new): the default heap is 25% of the container limit. How to check the limit the JVM detected, and a
    warning about JDK builds that don't detect it on kernels without cgroup v1 memory support (observed with Temurin
    24.0.2).

Why it does that

Briefly state why the change was necessary.

Further notes

List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.

Who will sponsor this feature?

Please @-mention the committer that will sponsor your feature.

Linked Issue(s)

Closes # <-- insert Issue number if one exists

Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

@wolf4ood
wolf4ood marked this pull request as ready for review October 30, 2026 08:31
@wolf4ood
wolf4ood requested a review from a team as a code owner October 30, 2026 08:31

@ndr-brt ndr-brt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great stuff!

@wolf4ood
wolf4ood merged commit 9981927 into eclipse-edc:main Oct 30, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants