Skip to content

Conversation

jpenilla
Copy link
Member

@jpenilla jpenilla commented Sep 17, 2025

Tracking issue: #12563

@github-project-automation github-project-automation bot moved this to Awaiting review in Paper PR Queue Sep 17, 2025
@jpenilla jpenilla linked an issue Sep 17, 2025 that may be closed by this pull request
9 tasks
jpenilla and others added 24 commits September 16, 2025 19:34
We don't need to scale this anymore as vanilla has per-level world borders
…rSpawnLocationEvent (for removal)

The Conversation API has been left to rot for the past few years and is considered dead. It does not support components and has had functionality ripped out over the years. Because of this, it is being deprecated for removal.

The metadata API is extremely misleading and causes memory leaks on every server I have seen use it. It just promotes bad practices and is misleading. The PDC system and just having their own solutions is better for developers.

Lastly deprecate PlayerSpawnLocationEvent for removal as events using this hack will be aggressively removed as we move towards better supporting the config API.
…At calls for loaded chunks' patch to code replaced by Moonrise
…ata to disk if it serializes without throwing'
Validate will be removed, and the methods in Objects are a
suitable replacement.
The old method of tracking the counts was to copy a
Long2IntOpenHashMap each time a count was removed or added. This
was under the assumption that tickets with counter types were not
being added or removed frequently.

The new system uses ConcurrentLong2LongChainedHashTable instead
to support concurrently updating ticket counters for different
positions.
We copy the map when retrieving the forced chunks, however this
appears to only be used in the ForceLoad command so this shouldn't
be an issue.

Additionally, the ticket counts were not updated if the ticket
were to expire or if the ticket was UNKNOWN and added during
ChunkHolderManager#removeTicket(). These issues are now resolved.
@PaperMC PaperMC locked as spam and limited conversation to collaborators Oct 1, 2025
@jpenilla jpenilla marked this pull request as ready for review October 3, 2025 00:18
@jpenilla jpenilla requested a review from a team as a code owner October 3, 2025 00:18
@jpenilla jpenilla merged commit a3f247c into main Oct 3, 2025
8 of 9 checks passed
@jpenilla jpenilla deleted the dev/1.21.9 branch October 3, 2025 00:39
@github-project-automation github-project-automation bot moved this from Awaiting review to Merged in Paper PR Queue Oct 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

1.21.9 Update Tracking