Skip to content

market: require indexing offsets and include preceding MK20 pieces - #1537

Open
hyunmoon wants to merge 2 commits into
filecoin-project:mainfrom
hyunmoon:codex/ten-pr-q4-20260913
Open

hyunmoon wants to merge 2 commits into
filecoin-project:mainfrom
hyunmoon:codex/ten-pr-q4-20260913

Conversation

@hyunmoon

@hyunmoon hyunmoon commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Prevent missing offsets from blocking Indexing and correct MK20 preceding-piece arithmetic.

  • Require non-NULL sector_offset in MK12/MK20 candidate selection and conditional assignment. Zero remains valid, and an older NULL row cannot hide a later ready row. Preserve provider/deal/aggregate identity and metadata-only behavior.
  • Advance the offset past every preceding padded piece, including alignment, in authoritative piece_index order.

No metadata fallback, backfill, task recovery or completion-policy change.

Validation and limits

Normal/race production-helper tests and an arithmetic red/green regression. PostgreSQL tests exercise both market versions, all-NULL/readiness cases, rollback, and candidate-guard removal followed by a passing restored baseline. These SQL fixtures have not run on Yugabyte. Arithmetic evidence does not depend on native/CUDA or long-duration sealing tests.

Series overview. Independently reviewable; no dependency on the foundation PR.

Henry added 2 commits September 12, 2026 06:06
Consolidated topic C04. Preserve the reviewed implementation and regressions; historical local execution receipts remain outside the source tree.

Source commits:
266e76e
407e3ae
6099fdf
8e7b88a
Consolidated topic C10. Preserve the reviewed implementation and regressions; historical local execution receipts remain outside the source tree.

Source commits:
cb88625
@FilOzzy FilOzzy added this to FOC Sep 14, 2026
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FOC Sep 14, 2026
@hyunmoon
hyunmoon marked this pull request as ready for review September 14, 2026 10:38
@hyunmoon
hyunmoon requested review from a team and ZenGround0 as code owners September 14, 2026 10:38
@FilOzzy FilOzzy moved this from 📌 Triage to 🔎 Awaiting review in FOC Sep 14, 2026
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