Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Random stuff #185

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

Random stuff #185

wants to merge 13 commits into from

Conversation

GroobleDierne
Copy link

Random stuff and cleanups of my recent work :)

@GroobleDierne
Copy link
Author

@nikosgram I think you can add what I put in 8b51eee in the release changelog

@GroobleDierne
Copy link
Author

Small question, I feel like this line is dead code, I don't see in which situation the check would succeed.

@nikosgram nikosgram marked this pull request as ready for review August 30, 2024 19:06
@nikosgram
Copy link
Owner

Could we potentially do the migration on the startup of the plugin?

@GroobleDierne
Copy link
Author

Could we potentially do the migration on the startup of the plugin?

I tried but ebean crashes as soon as the connection is opened. It could have been possible if the part of EBean that handles the migration was setup previously but it is not the case (I'm not blaming you I don't think it even existed in the version of EBean that was previously used lol)

@GroobleDierne GroobleDierne marked this pull request as draft August 30, 2024 19:28
@GroobleDierne
Copy link
Author

GroobleDierne commented Aug 30, 2024

I'll leave this as draft for now I still have things to upstream

@nikosgram nikosgram marked this pull request as ready for review August 30, 2024 23:08
@GroobleDierne
Copy link
Author

GroobleDierne commented Aug 30, 2024

@GroobleDierne
Copy link
Author

Now that chunk loading is no longer a problem I found other bottlenecks and got rid of them.
That is some juicy performance improvements 😋
Profiling was done when doing /t claim:
Before: https://spark.lucko.me/Mh6i8TPvgo
(3164ms spent in Gringotts)
After: https://spark.lucko.me/cMCGuQPNA5
(16ms spent in Gringotts and I had to lower the lag threshold to be able to capture something)


### ⚠ Breaking Changes in Version 3.0.0

This version changed the way data are stored, so if you come from an older version of Gringotts, you need to manually update your database.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: way data are stored --> data is stored.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants