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

Improve World DB Startup Caching #3587

Merged
merged 1 commit into from
Jun 12, 2021

Conversation

Mag-nus
Copy link
Member

@Mag-nus Mag-nus commented Jun 12, 2021

This is significantly faster than what is currently in master, and just slightly slower than PR #3582

It is now a single threaded operation, just like #3582 but instead of stitching the objects together manually, it lets EF stitch the objects together.

EF Tracking is required for the stitching to work.

This is the recommended pattern for these types of actions when dealing with EF.

I think similar improvements can be made to startup player caching

This is significantly faster than what is currently in master, and just slightly slower than PR ACEmulator#3582

It is now a single threaded operation, just like ACEmulator#3582 but instead of stitching the objects together manually, it lets EF stitch the objects together.

EF Tracking is required for the stitching to work.

This is the recommended pattern for these types of actions when dealing with EF.

I think similar improvements can be made to startup player caching
@gmriggs
Copy link
Collaborator

gmriggs commented Jun 12, 2021

measured on 2600k:

master: ~300s
#3587: 18s
#3582: 5s

~15x improvement from master, but still 3-4x slower than #3582

still significantly better than master, so marking as approved

@Mag-nus Mag-nus merged commit 7a0404c into ACEmulator:master Jun 12, 2021
@Mag-nus Mag-nus deleted the ImproveWorldDbCaching branch June 12, 2021 19:15
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.

None yet

2 participants