Skip to content

feat: add configurable table page sizes#1183

Draft
MichaReiser wants to merge 1 commit into
salsa-rs:masterfrom
MichaReiser:configurable-page-sizes
Draft

feat: add configurable table page sizes#1183
MichaReiser wants to merge 1 commit into
salsa-rs:masterfrom
MichaReiser:configurable-page-sizes

Conversation

@MichaReiser

Copy link
Copy Markdown
Contributor

Allow input, interned, and tracked structs to select 128-, 256-, 512-, or 1024-slot table pages while retaining 1024 as the default.
Smaller pages reduce unused slot capacity for sparse ingredients, while the tagged ID layout keeps mixed page sizes safe in one database.
Testing: Passed formatting, strict Clippy, full Nextest, no-default-feature, docs, Shuttle, Miri, and benchmark-build checks.

@netlify

netlify Bot commented Jun 23, 2026

Copy link
Copy Markdown

Deploy Preview for salsa-rs ready!

Name Link
🔨 Latest commit f12797f
🔍 Latest deploy log https://app.netlify.com/projects/salsa-rs/deploys/6a441fe817c48d00087dff5f
😎 Deploy Preview https://deploy-preview-1183--salsa-rs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@MichaReiser MichaReiser force-pushed the configurable-page-sizes branch 2 times, most recently from c108b8a to fd48a93 Compare June 24, 2026 07:22
@codspeed-hq

codspeed-hq Bot commented Jun 24, 2026

Copy link
Copy Markdown

Merging this PR will improve performance by 6.62%

⚡ 1 improved benchmark
✅ 10 untouched benchmarks
🗄️ 22 archived benchmarks run1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
WallTime hot_cache_hits[1024] 163.2 µs 153.1 µs +6.62%

Tip

Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.


Comparing MichaReiser:configurable-page-sizes (f12797f) with master (3084912)

Open in CodSpeed

Footnotes

  1. 22 benchmarks were run, but are now archived. If they were deleted in another branch, consider rebasing to remove them from the report. Instead if they were added back, click here to restore them.

@MichaReiser MichaReiser force-pushed the configurable-page-sizes branch from fd48a93 to f12797f Compare June 30, 2026 19:58
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.

1 participant