Critical fix: a bizarre Bukkit bug was resulting in the fill command sometimes wiping out the chunk at 0,0, even when it never even touched that chunk. The bug is caused by World.unloadChunk() in unknown circumstances. This workaround fix causes the task to speed along faster and memory usage to grow very quickly if you're running the fill task at a high frequency, so I've increased the "too low" memory threshold to 200 MB and increased the status update frequency and memory check to every 5 seconds (instead of every 10 seconds).

The status update frequency for the trim task has likewise been switched to every 5 seconds, for consistency.
This commit is contained in:
Brettflan 2011-08-15 19:04:02 -05:00
parent 933f9cac16
commit 548fbef7d6
3 changed files with 11 additions and 10 deletions

1
.gitignore vendored
View File

@ -5,3 +5,4 @@
/build
/dist
/lib/GroupManager.jar
/nbproject/pmd.settings

View File

@ -134,8 +134,8 @@ public class WorldFillTask implements Runnable
if (paused)
return;
// every 10 seconds or so, give basic progress report to let user know how it's going
if (Config.Now() > lastReport + 10000)
// every 5 seconds or so, give basic progress report to let user know how it's going
if (Config.Now() > lastReport + 5000)
reportProgress();
// if we've made it at least partly outside the border, skip past any such chunks
@ -172,10 +172,10 @@ public class WorldFillTask implements Runnable
{
CoordXZ coord = storedChunks.remove(0);
if (!originalChunks.contains(coord))
world.unloadChunk(coord.x, coord.z);
world.unloadChunkRequest(coord.x, coord.z);
coord = storedChunks.remove(0);
if (!originalChunks.contains(coord))
world.unloadChunk(coord.x, coord.z);
world.unloadChunkRequest(coord.x, coord.z);
}
// move on to next chunk
@ -257,6 +257,7 @@ public class WorldFillTask implements Runnable
// for successful completion
public void finish()
{
this.paused = true;
reportProgress();
world.save();
sendMessage("task successfully completed!");
@ -285,7 +286,7 @@ public class WorldFillTask implements Runnable
{
CoordXZ coord = storedChunks.remove(0);
if (!originalChunks.contains(coord))
world.unloadChunk(coord.x, coord.z);
world.unloadChunkRequest(coord.x, coord.z);
}
}
@ -334,7 +335,6 @@ public class WorldFillTask implements Runnable
// try to keep memory usage in check and keep things speedy as much as possible...
world.save();
server.savePlayers();
}
// send a message to the server console/log and possibly to an in-game player
@ -347,7 +347,7 @@ public class WorldFillTask implements Runnable
if (notifyPlayer != null)
notifyPlayer.sendMessage("[Fill] " + text);
if (availMem < 100)
if (availMem < 200)
{ // running low on memory, auto-pause
pausedForMemory = true;
Config.StoreFillTask();

View File

@ -120,8 +120,8 @@ public class WorldTrimTask implements Runnable
if (paused)
return;
// every 10 seconds or so, give basic progress report to let user know how it's going
if (Config.Now() > lastReport + 10000)
// every 5 seconds or so, give basic progress report to let user know how it's going
if (Config.Now() > lastReport + 5000)
reportProgress();
if (regionChunks.isEmpty())