Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add '-Ddd.resolver.type.pool.size' to control maximum size of resolver's type pool #3513

Merged
merged 1 commit into from
Apr 20, 2022

Conversation

mcculls
Copy link
Contributor

@mcculls mcculls commented Apr 20, 2022

What Does This Do

Controls how many type definitions are kept in the resolver's type pool before it starts evicting old entries.

Motivation

The default pool size of 64 (per-classloader) is likely too small for large enterprise applications leading to class resources repeatedly being requested, slowing down startup. Providing a flag to control its size allows users to accept a larger pool to reduce startup costs with the trade-off being potentially increased memory use.

Additional Notes

The equivalent environment variable is DD_RESOLVER_TYPE_POOL_SIZE

@mcculls mcculls added the tag: performance Performance related changes label Apr 20, 2022
@mcculls mcculls requested a review from a team as a code owner April 20, 2022 11:37
@mcculls mcculls force-pushed the mcculls/configurableTypePoolSize branch 3 times, most recently from 0737cd7 to 3b8c554 Compare April 20, 2022 17:11
@mcculls mcculls force-pushed the mcculls/configurableTypePoolSize branch from 3b8c554 to 4b38e89 Compare April 20, 2022 18:15
@mcculls mcculls merged commit 120dccf into master Apr 20, 2022
@mcculls mcculls deleted the mcculls/configurableTypePoolSize branch April 20, 2022 18:48
@github-actions github-actions bot added this to the 0.100.0 milestone Apr 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tag: performance Performance related changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants