Dynmap-Forge/Fabric

Dynmap-Forge/Fabric

888k Downloads

Folia Support

geferon opened this issue · 8 comments

commented

Feature Description:
Currently the PaperMC dev team is working on a Paper fork called Folia which adds support for Multi-threaded minecraft.
The issue with this is that, this obviously breaks some plugins, so their solution for it is making it so only plugins that have support for Folia can run in Folia (with a property in the plugin.yml file).
Some plugins are supported out of the box whereas others need some refactoring to be able to be supported.
I'm not sure of where Dynmap stands between these two, but nonetheless it would be nice to have suport for this in the future.

I can try to see how well it works with Folia or if Dynmap breaks with Folia, and perhaps also do a PR if necessary.

Folia: https://github.com/PaperMC/Folia

commented

Once Folia is released (GA, not beta), we'll evaluate supporting it. I think it is very likely that I will support it - but no commitments until I see both the released product and the resulting final API and model changes: Dynmap also has an API, and your intended changes will also have a nontrivial effect on that (since anyone calling us is now not necessarily doing so from the singular server thread, even if the command/call being issued is relevant to data on another world or on a different region of the same world...), so supporting Folia is also about supporting an API suitable to other Folia-enabled plugins for us.

Please do NOT do a PR - until I've done the evaluation to decide whether we'll support it, there is no way I'll accept it - and doing that evaluation will involve the design work that will effect the implementation across all platforms (since I'm NOT making the 95% common code for Dynmap split for the sake of one new platform...).

commented

Dynmap is currently not compatible with folia, probably because our chunkloading is not multihread capable, feel free to see if you can get it to work and create a PR!

commented

Alrighty, makes sense! Should I leave this issue open until then?

commented

Sure - it's a good placeholder, and it'll hopefully avoid folks opening additional ones.... (not counting on it, but it cannot hurt!)

commented

Any updates?

commented

No plans at this time - will be evaluated sometime after initial Folia GA/supported release.

commented

Folia looks extremely promising. I do hope you guys look into supporting it.

commented

Any updates about Folia support?