jusdino / BuildPortals

Mincreaft plugin for player-buildable teleportation portals
https://dev.bukkit.org/projects/buildportals
GNU General Public License v3.0
10 stars 5 forks source link

1.17 Incompatibility #31

Closed Tetrino closed 2 years ago

Tetrino commented 3 years ago

I wish I could submit an error log for you but the server and its related logs fill up, wipe out and crash.

Seems having the addon enabled in 1.17 fills the console with errors of some description, resulting in a lot of sever processing lag. I'll be updating this post with logs as soon as I can get them, but flagging the issue early in case you get chance to spin up a server ahead of time.

Thank you for this addon BTW, it's one of the favourites on my little community server!

Edit: Managed to get the dump.

[14:21:49 ERROR]: Current Thread: Server thread
[14:21:49 ERROR]:       PID: 23 | Suspended: false | Native: false | State: TIMED_WAITING
[14:21:49 ERROR]:       Stack:
[14:21:49 ERROR]:               java.base@16.0.1/jdk.internal.misc.Unsafe.park(Native Method)
[14:21:49 ERROR]:               java.base@16.0.1/java.util.concurrent.locks.LockSupport.parkNanos(LockSupport.java:252)
[14:21:49 ERROR]:               app//net.minecraft.util.thread.BlockableEventLoop.waitForTasks(BlockableEventLoop.java:144)
[14:21:49 ERROR]:               app//net.minecraft.util.thread.BlockableEventLoop.managedBlock(BlockableEventLoop.java:133)
[14:21:49 ERROR]:               app//net.minecraft.server.level.ServerChunkCache.getChunk(ServerChunkCache.java:516)
[14:21:49 ERROR]:               app//net.minecraft.world.level.Level.getChunk(Level.java:418)
[14:21:49 ERROR]:               app//net.minecraft.world.level.Level.getBlockState(Level.java:753)
[14:21:49 ERROR]:               app//org.bukkit.craftbukkit.v1_17_R1.block.CraftBlock.getNMS(CraftBlock.java:79)
[14:21:49 ERROR]:               app//org.bukkit.craftbukkit.v1_17_R1.legacy.CraftEvil.getType(CraftEvil.java:47)
[14:21:49 ERROR]:               frahm.justin.mcplugins.buildportals.PortalHandler.integrityCheck(PortalHandler.java:68)
[14:21:49 ERROR]:               frahm.justin.mcplugins.buildportals.PortalHandler.checkAndUpdate(PortalHandler.java:99)
[14:21:49 ERROR]:               frahm.justin.mcplugins.buildportals.BPListener$2.run(BPListener.java:148)
[14:21:49 ERROR]:               app//org.bukkit.craftbukkit.v1_17_R1.scheduler.CraftTask.run(CraftTask.java:100)
[14:21:49 ERROR]:               app//org.bukkit.craftbukkit.v1_17_R1.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:468)
[14:21:49 ERROR]:               app//net.minecraft.server.MinecraftServer.tickChildren(MinecraftServer.java:1488)
[14:21:49 ERROR]:               app//net.minecraft.server.dedicated.DedicatedServer.tickChildren(DedicatedServer.java:477)
[14:21:49 ERROR]:               app//net.minecraft.server.MinecraftServer.tickServer(MinecraftServer.java:1404)
[14:21:49 ERROR]:               app//net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1180)
[14:21:49 ERROR]:               app//net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:320)
[14:21:49 ERROR]:               app//net.minecraft.server.MinecraftServer$$Lambda$3675/0x000000080138e390.run(Unknown Source)
[14:21:49 ERROR]:               java.base@16.0.1/java.lang.Thread.run(Thread.java:831)

Looks like we're ending up in an infinite loop (we're seeing hundreds of these). Could be because 1.17 is still being worked on, but worth flagging.

NieldSWND commented 3 years ago

Thanks for reporting! Starting work on compatibility patches now. Will hopefully have an update for you in the next week.

Tetrino commented 3 years ago

Thanks for taking a look - take your time!