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

Use optimized checksumming when loading database #441

Merged
merged 3 commits into from
Jan 9, 2025
Merged

Conversation

btoews
Copy link
Member

@btoews btoews commented Jan 8, 2025

I wanted to make sure we like the new API before merging superfly/ltx#56, so this PR targets that branch currently. If we're happy with how this PR looks, I'll merge the ltx one and rebase this one.

Update: I merged and cut a new release for superfly/ltx#56. I rebased this branch, so it should be ready to go.

@btoews btoews requested a review from benbjohnson January 8, 2025 19:22
@btoews
Copy link
Member Author

btoews commented Jan 8, 2025

Looks like CI is broken

Run sudo apt install -y fuse3 libsqlite3-dev consul

WARNING: apt does not have a stable CLI interface. Use with caution in scripts.

Reading package lists...
Building dependency tree...
Reading state information...
Package consul is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

E: Package 'consul' has no installation candidate

Update: Fixed it

@btoews
Copy link
Member Author

btoews commented Jan 8, 2025

I merged and cut a new release for superfly/ltx#56. I rebased this branch, so it should be ready to go.

@btoews btoews marked this pull request as ready for review January 8, 2025 20:48
@btoews
Copy link
Member Author

btoews commented Jan 9, 2025

I'm seeing the hoped-for boot speed improvements running this branch in production 🎉

@btoews btoews merged commit 0f9f2bf into main Jan 9, 2025
16 checks passed
@btoews btoews deleted the checksum-perf branch January 9, 2025 15:37
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.

2 participants