Skip to content

Bump zstd-seekable-format-go to v0.10.0#38

Merged
jtarchie merged 2 commits into
jtarchie:mainfrom
SaveTheRbtz:upgrade/zstd-seekable-format-go-v0.10.0-rc.1
Jun 4, 2026
Merged

Bump zstd-seekable-format-go to v0.10.0#38
jtarchie merged 2 commits into
jtarchie:mainfrom
SaveTheRbtz:upgrade/zstd-seekable-format-go-v0.10.0-rc.1

Conversation

@SaveTheRbtz

@SaveTheRbtz SaveTheRbtz commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

What changed

Bumps github.com/SaveTheRbtz/zstd-seekable-format-go/pkg from v0.8.0 to v0.10.0: https://github.com/SaveTheRbtz/zstd-seekable-format-go/releases/tag/pkg%2Fv0.10.0

This keeps sqlitezstd compiling against the stable v0.10 API while preserving the existing VFS behavior.

API migration

Validation

  • go test -count=1 ./...

@SaveTheRbtz SaveTheRbtz marked this pull request as ready for review June 3, 2026 22:53
@SaveTheRbtz

Copy link
Copy Markdown
Contributor Author

JFYI: there is also support for configurable cache on the zstd-seekable-format-go side, so you can remove up to 1K lines of code:

There are multiple levels of cache removals in the PR, you can pick a subset of commits

  • compressed blocks cache
  • decompressed blocks cache
  • http request cache

Having one cache benefits non-infinite memory scenarios where we do not want to duplicate the same data in 3 different caches.

I've ran basic perf tests with linux kernel sources sqlite database with fts5 index served over http. A single Sieve cache has the +/- same performance as 3 separate ones after we control for memory usage

@SaveTheRbtz SaveTheRbtz changed the title Bump zstd-seekable-format-go to v0.10.0-rc.1 Bump zstd-seekable-format-go to v0.10.0 Jun 4, 2026
@jtarchie

jtarchie commented Jun 4, 2026

Copy link
Copy Markdown
Owner

I will take a look, thanks for the work.

I'm just working on loge to support some more efficient work reading over HTTP. So a product was driving out the feature. I'll see if I can use that to benchmark.

I see you have an upstream-upstream PR. Should I merge this first, then will you add the other one?

@jtarchie jtarchie merged commit 8732b5f into jtarchie:main Jun 4, 2026
1 check passed
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