webbukkit / Dynmap-Factions

Display faction
Display Factions regions on Dynmap's maps, with live update
6 stars 17 forks source link

TPS drops on radius claim and unclaim #11

Closed oloflarsson closed 10 years ago

oloflarsson commented 11 years ago

When many people radius claim or radius unclaim large areas the TPS drops to 0 for a short while. We just released a new world map on my server and due to all the claming the profiling results looks like this: http://i.imgur.com/7S3TDEO.png

Could you optimize this somehow? Perhaps reduce the ordo some way. Additionally you could possibly run the update async instead?

oloflarsson commented 11 years ago

I think this would be the line getting repeated to often: https://github.com/webbukkit/Dynmap-Factions/blob/1d71852a81ef5ce9de87d138f32c3354af5d21f6/src/main/java/org/dynmap/factions/DynmapFactionsPlugin.java#L476

oloflarsson commented 10 years ago

bump?

mikeprimm commented 10 years ago

Not sure which version you're running - give http://mikeprimm.com/dynmap/builds/Dynmap-Factions/Dynmap-Factions-0.72.jar or http://mikeprimm.com/dynmap/builds/Dynmap-Factions-1.8.x/Dynmap-Factions-1.8.x-0.72.jar , as appropriate

oloflarsson commented 10 years ago

This is the plugin.yml

name: Dynmap-Factions
main: org.dynmap.factions.DynmapFactionsPlugin
version: "0.71-26"
author: mikeprimm
depend: [ dynmap, Factions ]
mikeprimm commented 10 years ago

Need 0.72 for fix - now released