Geolykt / EnchantmentsPlus

Enchantments+, a Bukkit Custom Enchantments Plugin based on Zenchantments targetted at production grade servers
GNU General Public License v3.0
38 stars 15 forks source link

Spectral can crash the server #8

Closed Geolykt closed 4 years ago

Geolykt commented 4 years ago

Affected enchantment(s):

Main symptom(s):

Secondary symptom(s):

Possible resolution(s):

Additional notes:

Due to the high and rapid spike of Memory usage, it is likely that we are dealing with the fact that the plugin cannot handle massive amounts of data quickly and that a very high amount of objects are created, fixing it might be performed by recycleing objects, more specificially the permission query object.

Geolykt commented 4 years ago

It should be mitigated, however I must stress that this is mitigated, so not much performance improvements. I'd likely conduct polls on how the enchantment is used to be able to optimise it further, but from what I see I can only optimise it by rearraging the if-else blocks to prioritize the most likely senario