search

apavarino / Deadchest

Minecraft Plugin - Keep your inventory in a chest when you die !
https://dev.bukkit.org/projects/dead-chest
GNU General Public License v3.0
18 stars 17 forks source link

High CPU usage with DeadChest loaded #36

Closed kccricket closed 2 years ago

kccricket commented 2 years ago

Context

Describe the bug When DeadChest is enabled, the server has extremely high CPU usage. While the server was running, I used PlugMan to unload DeadChest, and CPU usage immediately returned to normal. This was a fresh upgrade from Paper 1.17.1 to 1.18.1 and from DeadChest 4.7.0 to 4.8.1.

To Reproduce Unknown.

Timings: https://timings.aikar.co/?id=1f398f40cbd840398a356c65d2ac6033

It looks like the majority of the tick utilization is in "Minecraft::survive - Sync Chunk Load (SELF)". Oddly, no players were in the 'survive' world at the time, so I'm not sure what it was doing. deadchest.log doesn't contain any lines for the time period.

config.yml.txt chestData.yml.txt

assada commented 2 years ago

same issue https://timings.aikar.co/?id=5553b0035980478fa4ab8a49c3042f15

apavarino commented 2 years ago

Fixed on 4.8.2 !

You can download it on discord or wait for bukkit validation staff

assada commented 2 years ago

https://timings.aikar.co/?id=080a92765c284022bbc8a57b0026a34b#timings

nothing changed

apavarino commented 2 years ago

On Deadchest.log have you expired time spam message every sec ?

assada commented 2 years ago

No. Deadchest.log looks ok without any spam