Skip to content

Don’t retain the parsed node tree after loading a YamlConfiguration - #14436

Open
bobhenl wants to merge 1 commit into
PaperMC:mainfrom
bobhenl:fix/yaml-constructor-cache
Open

bobhenl wants to merge 1 commit into
PaperMC:mainfrom
bobhenl:fix/yaml-constructor-cache

Conversation

@bobhenl

@bobhenl bobhenl commented Oct 30, 2026

Copy link
Copy Markdown

YamlConfiguration keeps its YamlConstructor, and SnakeYAML’s constructor caches every constructed node with its marks until the next load. That keeps the full node tree alive for every loaded config. The caches are now cleared after fromNodeTree.

@bobhenl
bobhenl requested a review from a team as a code owner October 30, 2026 18:17
@github-project-automation github-project-automation Bot moved this to Awaiting review in Paper PR Queue Oct 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Awaiting review

Development

Successfully merging this pull request may close these issues.

1 participant