Multiverse-Inventories

Bukkit Plugins
878k Downloads

Inventories causing crashes when a player tries to teleport

trickman325 opened this issue ยท 19 comments

commented

Running paper 1.17.1 - 112
with Multiverse-inventories 4.2.2
and Lands 5.12.1

When a player tries to teleport every so often with /lands teleport it crashes the server

This is the log from the most recent crash
2021-07-21-2.log

Ive contacted the Lands dev and there is nothing he can do as it is Inventories and paper related not his side of things.

Really need to get this sorted as its casing issues when even when the server isnt stressed

commented

Lands.zip
Multiverse-Core.zip
Multiverse-Inventories.zip

These are my files in which this error happened @nicegamer7 @benwoo1110

commented

@benwoo1110 any news on this???
This is becoming a major issue for my server :/

commented

Lands is behind paid walls, so very hard for me to have anything to work off.

commented

I've contacted Angeschossen and am waiting a response to hopefully give you access

commented

@benwoo1110 ^^

commented

If you could send me your spigot account, I could give you access to the download.

Lands pre-loads the destination chunk via the chunk tickets before teleporting to the destination. After the teleportation is done, it will release the chunk again. I've made sure that synchronization is enforced at the usage of the chunk tickets. Also the issue is only existent on paper and their forks.

commented

ben will be busy this week, so I'll try to take a look. My spigot username is nicegamer7 and ben's is benji_0224.

commented

@Angeschossen ^^

commented

ben will be busy this week, so I'll try to take a look. My spigot username is nicegamer7 and ben's is benji_0224.

Added both.

commented

[21:20:19] [Server thread/WARN]: Can't keep up! Is the server overloaded? Running 5001ms or 100 ticks behind

this warning was shown around 5 to 10 times before the crash. Likely your server is just lagging and just nice it timed out at mvinv operations.

commented

@benwoo1110 That lag was over an hour before the crash. There was only 5 people on the server when it crashed and it was caused by the command running which you can see in that log at the bottom.

Here are other logs that are the exact same issue after restarts.
2021-07-20-3.log
2021-07-20-2.log

This was also what the Lands dev Angeschossen had to say about it

  • Looks like "com.onarandombox.multiverseinventories" doesn't like papers async player teleportation. This is where is crashes. It listens to the teleport called by the server when lands teleported a player.
    I'll see if I can do something, but most likely this is only fixable on "com.onarandombox.multiverseinventories" side.
    But I highly doubt it will change something as the crash does happen at the inventory plugin.
commented

Looks like this issue is specific to only when teleport is done by land plugin. Using essentials teleport commands or mvtp doesnt result in such as issue from my testing. As such I will need a link to their plugin and hope it's not behind closed doors.

commented

So it was a bug on your end?

commented

This ticket can be closed. We found a solution.

commented

It looks like it's a problem with the combination paper + chunk plugin tickets or I did something wrong on my end. I just changed the way how the teleportation works without using chunk plugin tickets. Issue resolved.

Although the server would only crash at using paper and multiverse-inventories installed.

commented

Glad to hear that it's fixed... but very interesting why mv-inv specifically tho, sadly I have no time to debug Minecraft things atm

commented

Quite strange, yeah. You might want to report this issue to Paper, since you said it only happens with Paper. I don't think MV does anything out of the ordinary regarding chunks.