-
Notifications
You must be signed in to change notification settings - Fork 75
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
Native transport does not work after Java 8 #58
Comments
That 6 year old netty would not support Java 9 without native transport (at least will not take advantages of it), too. Why not just use Java 8? I know later versions have many improvements but you are using an old Minecraft version, what you expect? If you're a latest bleeding-edge guy, then why not just use 1.16.5? If most of the patches, important fixes and such weren't backported to Java 8 maybe, but Java 8 is still supported and updated by Oracle/OpenJDK. Besides, MC officialy switched to Java 8 on MC 1.12 too.. 1.8 was probably compiled with Java 6 and Spigot/Paper has reflection errors on Java 9+. Perhaps they are fixed in SportPaper, but that doesn't mean it will work out of the box with full Java 15 support or such. Keeping up to date every dependency like Netty on a 8ish year old code base is not easy. |
Netty is good with backwards-compatibility, so its possible we could bump it. |
CobbleSword/NachoSpigot@0bd2d92 Might be worth looking into this, though I'm not sure if you need to make that high of a jump (potentially introducing more incompatibilities), since I haven't tested it. |
That NachoSpigot patch breaks a lot of plugins that expects older versions in 1.8.8, like ProtocolLib / Citizens, i gived a fix to ProtocolLib by doing a pr and tried the same with citizens but mcmonkey denied the pr because im a 'piracy supporter' |
This is possible, and has been proven by NachoSpigot:
|
After some testing, updating |
Then why does NachoSpigot have to patch the plugins for them to work.? Either way, if it truly doesn't break plugins I would love to see this added. New Java versions have a lot of performance improvements and some plugins are removing support for Java 8. |
I tested with NachoSpigot and official ViaVersion / ProtocolLib / etc versions and everything works fine, so it's possible to add support for newer Java Versions without patching other plugins. |
If you:
use-native-transport
set totrue
inserver.properties
which improves the performance on Linux.Then when you join the server, it will spam this Netty exception:
This issue is nothing new, but surely there's a way to fix it! Maybe just update Netty (the currently used version is about 6 years old)? The new versions of Java have a lot of performance improvements, that's why I would love to use them.
The text was updated successfully, but these errors were encountered: