DeadChest

DeadChest

187k Downloads

[BUG] ChestTask causes severe lag spikes (TPS drops) & folder name inconsistency

p5ych0p41h opened this issue · 1 comments

commented

Context

  • Plugin name: DeadChest
  • Plugin version: 4.22.1
  • Minecraft version: 1.20.6
  • Server type: Paper build 151

Describe the bug

  1. There are two issues with the latest version:
    Performance Issue: The plugin's scheduled task, me.crylonz.deadchest.tasks.ChestTask, is causing severe server lag spikes ("Can't keep up!" messages in console) for several seconds. A Spark profiler report clearly shows this task consuming a very high amount of the main server thread. This happens periodically, likely when the task runs to check on active chests. Spark Profiler Report: https://spark.lucko.me/3cfz0OaOAK
  2. Inconsistency Issue: On previous versions, the plugin's data folder was named DeadChest (with a capital C). On version 4.22.1, it now creates a folder named deadchest (all lowercase). This can cause confusion and issues with configuration files not being loaded from the old folder after an update.

To Reproduce
Steps to reproduce the behavior:

  1. Run the server with several players online.
  2. Wait for the ChestTask to run.
  3. Observe a "Can't keep up!" message in the console and a server-wide freeze in-game.
  4. The Spark profiler confirms that ChestTask is the cause.

Expected behavior
The ChestTask should run efficiently in the background without causing major TPS drops or freezing the server. The plugin folder name should remain consistent across updates to avoid configuration issues.

commented

Hello, thanks a lot for your report!
After checking your Spark report, I couldn’t find anything related to DeadChest:

Image

Regarding the name issue, you’re right — I’ll fix that in the next version. 👍