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

Fixes for large Servers #593

Merged
merged 1 commit into from
Oct 5, 2014
Merged

Fixes for large Servers #593

merged 1 commit into from
Oct 5, 2014

Conversation

slimshadeey1
Copy link
Contributor

One thing that is a constant problem is on multiplayer servers morphs is preventing them to startup after a while. As we know very well that NBT data files can be huge without problems, it can only mean one thing, the HashMap. It can be efficient to store all data in single player servers on a hash map until the server is restarted as you only have 1 - 5 players on. But this becomes problematic on large servers that have thousands of players. If all players joined throughout a day are kept loaded in the map even if they are not on, would result in a large amount of lag due to space required to hold everything in a map. A solution is to remove the player from the map on logout, as this prevents server owners from having to do constant restarts.

One thing that is a constant problem is on multiplayer servers morphs is preventing them to startup after a while. As we know very well that NBT data files can be huge without problems, it can only mean one thing, the HashMap. It can be efficient to store all data in single player servers on a hash map until the server is restarted as you only have 1 - 5 players on. But this becomes problematic on large servers that have thousands of players. If all players joined throughout a day are kept loaded in the map even if they are not on, would result in a large amount of lag due to space required to hold everything in a map. A solution is to remove the player from the map on logout, as this prevents server owners from having to do constant restarts.
iChun added a commit that referenced this pull request Oct 5, 2014
@iChun iChun merged commit 059302d into iChun:master Oct 5, 2014
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