search

Chicken-Bones / ForgeMultipart

An API for dynamically handling different functional parts in the one block space.
Other
119 stars 56 forks source link

TickScheduler memory/chunk leak #321

Open sfPlayer1 opened 9 years ago

sfPlayer1 commented 9 years ago

FMP version 1.7.10-1.1.3.127 references large quantities of chunks (~2 GB worth after 1 day) on FC2.

The few chunk objects I've checked were only referenced through codechicken.multipart.TickScheduler$WorldTickScheduler.chunkMap while the world reference in worldMap a few layers above is valid (referenced by DedicatedServer). Something about chunk unload handling isn't working properly.

I have a heap dump (1 GB compressed).

Chicken-Bones commented 9 years ago

Does that FMP version even exist? Source: http://files.minecraftforge.net/ForgeMultipart/

Edit: There's no chunkMap field either, perhaps tickChunks?

simon816 commented 9 years ago

That's a CCL version - http://files.minecraftforge.net/CodeChickenLib/

chunkMap is a field in the superclass, WorldExtension

Chicken-Bones commented 9 years ago

aah right, I misread that, I'll take a look

sfPlayer1 commented 9 years ago

Erm sorry, I derped. I wanted to file against CCL first, but forgot to update the version, the correct one is 1.7.10-1.1.1.323.