Skip to content

Use fastutil - #148

Open
kalibrieren wants to merge 30 commits into
Wind-Development:masterfrom
Falchus:use-fastutil
Open

Use fastutil#148
kalibrieren wants to merge 30 commits into
Wind-Development:masterfrom
Falchus:use-fastutil

Conversation

@kalibrieren

@kalibrieren kalibrieren commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

not tested but copied it over from FalchusSpigot/SportPaper so should work just fine

@windcolor-dev

Copy link
Copy Markdown
Member

We want to be careful not to break anything. Did you find these changes to boost performance meaningfully?

@kalibrieren

Copy link
Copy Markdown
Contributor Author

We want to be careful not to break anything. Did you find these changes to boost performance meaningfully?

👍

Comment thread WindSpigot-Server/src/main/java/net/minecraft/server/PlayerChunkMap.java Outdated
Comment thread WindSpigot-Server/pom.xml
Comment thread WindSpigot-API/pom.xml
@windcolor-dev

Copy link
Copy Markdown
Member

I'm curious, do you have like a profiler report or something to compare?

@kalibrieren

Copy link
Copy Markdown
Contributor Author

I'm curious, do you have like a profiler report or something to compare?

nope, but should be quite a bit faster vs trove/java.
excited to see how we can replace fastutil once java's project valhalla successfully gets implemented :)

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