MCPets Database syncing fix

Nocsy-Workshop / mcpets

MCPets plugin for Minecraft Spigot - Made by Nocsy | Partnered with MCModels.net

https://www.patreon.com/tofnocsy_workshop

GNU General Public License v3.0

56 stars 33 forks source link

MCPets Database syncing fix #21

Closed iRedFox closed 1 year ago

iRedFox commented 1 year ago

Replaced Hashmaps to ConcurrentHashMaps since they're thread-safe and internally synced.
Added a new method reloadAll(uuid) to reload specific player data and not all players.
Added a delay and changed the way login/quit logic is for saving playerdata
addExperience(double) now saves even if not able to level up to sync exp daa
Now Databases class uses locks to load/save to ensure there's no race-condition
And other changes/clean up

AlexandreChaussard commented 1 year ago

I'm gonna review that this weekend, sorry for the delay. I'll probably compile it and make sure the YAML system still works, and we'll debug the DB from the BETA channel with the testers 👍

Thanks for the fix