Skip to content
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

Player heads in vault and server freeze #372

Open
BoredFrostbite opened this issue Dec 12, 2018 · 10 comments
Open

Player heads in vault and server freeze #372

BoredFrostbite opened this issue Dec 12, 2018 · 10 comments

Comments

@BoredFrostbite
Copy link

BoredFrostbite commented Dec 12, 2018

PlayerVaultsX version: 4.0.6

Server software: 1.12.2

Details: Server sometimes freezes when a player use /pv [number], after checking the player vault, there were filled with heads (maybe the main cause?)

*I removed the paper watchdog log and timings because it was soooooo long (and pretty much not necessary).

@BoredFrostbite
Copy link
Author

BoredFrostbite commented Dec 12, 2018

crap, I just found it that 4.0.6 is only for 1.13, maybe it was the real cause, I will switch to 4.0.5 and leave this here in case it happens again.

@ArtelGG
Copy link
Contributor

ArtelGG commented Dec 15, 2018

If you could produce an error and perhaps show your plugins and a timings report I might be able to help at least identify what's causing it. Also, the changes in 4.0.6 should only be affecting vault signs.

@drtshock
Copy link
Owner

Can you give us the exact contents of the inventory, your playervaults .yml file that contains this player's inventory, log files showing this, or any info on how to reproduce exactly.

@ArtelGG
Copy link
Contributor

ArtelGG commented Dec 16, 2018

@drtshock

Can you give us the exact contents of the inventory, your playervaults .yml file that contains this player's inventory, log files showing this, or any info on how to reproduce exactly.

I have a sneaking suspicion this may be an issue with a player head plugin, perhaps one that allows the users to "redeem" them for money or something along those lines.

@BoredFrostbite
Copy link
Author

BoredFrostbite commented Dec 16, 2018

I checked that player's vault, vault number 3 is filled with player heads.
About reproducing the freeze, I tried several times but it won't freeze anymore, maybe because I used the wrong version before (4.0.6 for 1.12.2 instead of 4.0.5, something went wrong and the server frozen [althought you said that 4.0.6 only affects vault signs])
You can see the log and timings link by looking at the edit history (I don't mind pasting it back but its length is kinda annoying for me).

5b36d543-3dba-3c3d-857a-2162a64b90ea.zip

@ArtelGG
Copy link
Contributor

ArtelGG commented Dec 16, 2018

I did some testing on my test server with 1.8.8 and 1.12.2 PaperSpigot's, I didn't get any lag other than client end with 1.8.8, however, on 1.12.2 my server and client did hiccup a bit during the first opening of the vault. It might be an issue with how 1.9+ handles player heads.

@drtshock
Copy link
Owner

Yeah just tested that vault and it gave me a little bit of a delay in populating the heads in the inventory. Not sure what the best way to handle this would be..

@ArtelGG
Copy link
Contributor

ArtelGG commented Dec 17, 2018

You could try adding the items to a load-queue if they're an instance of a skull. Load them in packs of 5 or something.

@drtshock
Copy link
Owner

Perhaps, I would be concerned that we'd be open to dupes during that save window though.

@ArtelGG
Copy link
Contributor

ArtelGG commented Dec 31, 2018

You could mark them as in a vault, load the skulls, then open the inventory? It might add some delay but it'd most likely fix it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants