Skip to content
This repository has been archived by the owner on Dec 28, 2021. It is now read-only.

Usefull loaded chunks in timings #14

Open
wants to merge 30 commits into
base: master
Choose a base branch
from

Conversation

Maximvdw
Copy link

Adds total loaded chunks to timings (in comments)

md-5 and others added 30 commits July 30, 2014 03:09
Adds performance tracking timings all around the Minecraft Server, and improves the usability of the /timings command

Plugins can track their own timings with CustomTimingsHandler
Defines EntitySpawnEvent and SpawnerSpawnEvent. Adds BUKKIT-267 and BUKKIT-1559
- All of these changes have been reviewed to be binary compatible and in general contract compatible with previous versions of the libraries.
Add InventoryClickEvent.getClickedInventory. Adds BUKKIT-4495
Plugins currently have to do the logic themselves on the raw slot ID
in order to determine the inventory clicked. This provides the logic for plugins to
readily identify which inventory was clicked.
Most of these methods still have plenty of use given that only one player with each name can exist at a time. Deprecating these methods renders even basic functionality such as /msg <name> impossible without causing compiler warnings. We will maintain this API and it should be considered safe and appropriate for most use cases.
When on Java 7 we can register the classloader as parallel capable to prevent deadlocks caused by certain scenarios. Due to the nature of PluginClassLoader this isn't completely safe, but we can make it safer by switching to concurrency focused collections. Either way this is far better than crashing the server.
Use online players instead of offline players, which is very slow.
Adds total loading chunks to timings (in comments)
@sha-1 sha-1 force-pushed the master branch 8 times, most recently from ec88291 to 06f66f4 Compare August 30, 2014 17:41
@sha-1 sha-1 force-pushed the master branch 21 times, most recently from c1eb84d to b3e065a Compare September 4, 2014 09:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants