Project-Poseidon icon indicating copy to clipboard operation
Project-Poseidon copied to clipboard

Re add Chunk Cache as an option

Open I-Tom-I opened this issue 1 year ago • 5 comments

Please re add the Bukkit Chunk Cache as a server option in poseidon.yml as it helps performance. I've been on CB1060 for years and noticed it's better with world loading, especially if you have minecart tracks that go between places that cause chunks to be loaded/unloaded repeatedly. Uncommenting the lines commented out in ce48b13 and ce48b13 helps with the issue so adding ifs to toggle it via a setting would be great so we don't have to use a fork and need to compile every new version of Poseidon.

I-Tom-I avatar Mar 04 '23 03:03 I-Tom-I

I am not keen on adding an option to reenable the default chunk cache due to its significant exploits, allowing players to dupe, bypass permissions, etc. However, I would accept that PR if someone wanted to investigate the chunk cache and resolve the bugs.

I plan to add async chunk loading when I get more time, however, that is likely a while away.

RhysB avatar Mar 05 '23 00:03 RhysB

That is why it was suggested as a poseidon.yml option, the default can be off and those who want it can enable it. That way we don't need to fork/merge/recompile Poseidon every time a change is made to keep chunk cache on.

I-Tom-I avatar Mar 09 '23 19:03 I-Tom-I

I don't think this is really needed since Strultz backported a faster chunk loading fix from modern Minecraft versions. (#61 https://github.com/RhysB/Project-Poseidon/commit/3609ce2cced8cb00af58048dfbd0c3f3f93e3ea7)

ghost avatar May 09 '23 14:05 ghost

It's more of an issue when the hard drive is slow at giving the server data, especially at max view distance. Traveling between areas on minecarts can put stress on slower hard drives and players will experience slow world loading. Again I'm asking for the option, it doesn't need to be a default, because otherwise we'll have to just stick with CB1060.

I-Tom-I avatar May 11 '23 15:05 I-Tom-I

It's more of an issue when the hard drive is slow at giving the server data, especially at max view distance. Traveling between areas on minecarts can put stress on slower hard drives and players will experience slow world loading. Again I'm asking for the option, it doesn't need to be a default, because otherwise we'll have to just stick with CB1060.

Well I assume that would use a lot more RAM, and we all know this version of server software is not so good at memory management.

ghost avatar May 11 '23 16:05 ghost