Aikar f36773fd9c Use Last Access Cache for isLoaded calls ...

this should provide a pretty good across-the-board performance
improvement for the entire server, as it's very common for code
to check if (isLoaded(x, z)) { then do something on that chunk }

Previously, containsKey would not read or set the last access cache

By making it do get() != null, we gain the benefits of last access
and also improves thread safey for async isLoaded checks

This exact usage scenario is used in Entity movement, so that alone
saves us up to 5%~ of CPU time for Entity movement.

2018-10-23 23:50:51 -04:00

5.4 KiB

Raw Blame History

View Raw