Skip to content

Conversation

@Gozala
Copy link
Contributor

@Gozala Gozala commented Mar 5, 2025

Changes here remove complicated joins per @seefeldb findings that creates huge perf wins and same stuff loads goes from 13sec to 675ms on my machine.

I also added a migration scripts that can be used to migrate old db to new one by running it like

deno task migrate ./path/to/db.sqlite

@Gozala Gozala requested review from ellyxir and seefeldb March 5, 2025 04:51
Copy link
Contributor

@ellyxir ellyxir left a comment

Choose a reason for hiding this comment

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

looks good to me. reviewed code, ran migration script and it worked, ran common-memory and jumble and it brought down time from 10 sec to 3.4 seconds.

@Gozala Gozala merged commit edddb29 into main Mar 5, 2025
6 checks passed
@Gozala Gozala deleted the fix/memory-perf branch March 5, 2025 18:19
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