Skip to content

docs: more tuning guidance - #238

Merged
wolf4ood merged 1 commit into
eclipse-edc:mainfrom
wolf4ood:docs/tuning_replicas
Nov 2, 2026
Merged

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

Conversation

@wolf4ood

@wolf4ood wolf4ood commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

What this PR changes

More tuning guidance from the 1.0 load tests with several controlplane replicas:

  • Database connections: keep max-idle close to max-total. With the default of 4 the pool keeps closing and
    reopening connections under bursty load, which cost PostgreSQL about a third of its CPU in the load test.
  • Credential service (new): with DCP every incoming protocol message triggers a presentation query to the
    counter-party's credential service, so an IdentityHub serves the protocol traffic of all the participants it hosts.
    With its default pool of 8 connections it capped the whole platform.
  • Finding the limit: how a saturated credential service shows up in thread dumps.
  • Replicas need CPU of their own to add throughput.
  • Tracing has a measurable cost under load.

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 November 2, 2026 08:52
@wolf4ood
wolf4ood requested a review from a team as a code owner November 2, 2026 08:52
@wolf4ood
wolf4ood merged commit 0008028 into eclipse-edc:main Nov 2, 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