Skip to content

test(db): document the range count contract's dedicated stack - #1170

Merged
xav-db merged 2 commits into
mainfrom
bound-range-count-test-stack
Oct 6, 2026
Merged

xav-db merged 2 commits into
mainfrom
bound-range-count-test-stack

Conversation

@xav-db

@xav-db xav-db commented Oct 5, 2026 •

Copy link
Copy Markdown
Member

Main now includes the range-intersection count contract's 16 MiB stack fix through #1172. Merge current main to resolve the duplicate change and retain the explanation that the debug future must be constructed and polled on the dedicated thread. The remaining diff is two comment lines; the fixture and every assertion match main.

libtest → dedicated 16 MiB thread → Tokio → unchanged contract

Validated the merged branch on fresh ARM64 EC2:

  • All 58 production contracts passed, plus the count contract with a 1 MiB outer test stack.
  • Workspace Clippy (-D warnings) and formatting passed.
  • Reviewed the diff and confirmed it changes only the explanatory comment.

The original fix was also validated with rebuilt-image packaging, native-volume persistence, S3/Compose suites, full workspace tests and doctests before main incorporated it.

@matthewsanetra matthewsanetra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same fix as #1171. Workspace tests and DB all targets pass; the two red jobs are time-limit cancellations (Clippy after every test step passed; DB coverage, which times out on every PR since the async index queue).

@matthewsanetra

Copy link
Copy Markdown
Contributor

Superseded: main already has the same change (92c1575, via #1172), and #1173 raised the quality job's time limit. Thanks for the Linux validation; it matched ours.

@matthewsanetra matthewsanetra reopened this Oct 6, 2026
@xav-db xav-db changed the title test(db): run range count contract on its dedicated stack test(db): document the range count contract's dedicated stack Oct 6, 2026
@xav-db
xav-db merged commit ffc44a9 into main Oct 6, 2026
30 of 31 checks 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