Skip to content

feat(stovepipe): Add fact degree lookup table - #908

Merged
prathshenoy merged 1 commit into
mainfrom
feat/stovepipe-project-order-index
Oct 23, 2026
Merged

prathshenoy merged 1 commit into
mainfrom
feat/stovepipe-project-order-index

Conversation

@prathshenoy

@prathshenoy prathshenoy commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Summary

What:

  • Maintain a degree-ordered lookup table for project validation facts and repair interrupted writes on retry.
  • Start the lookup table empty while keeping existing project listing and pagination unchanged.

Why:

  • Prepare efficient severity-first reads without historical migration or backfill tooling.

Test Plan

  • Add unit tests.
  • Verify database writes and unchanged service and browser pagination against real services.

Revert Plan

  • Revert dependent readers first, then this PR. Leave the additive table in place; authoritative validation facts are unchanged.

Issue Links

Stack

  1. feat(stovepipe): Add fact degree lookup table #908
  2. feat(stovepipe): Show most broken projects first #903

@prathshenoy
prathshenoy marked this pull request as draft October 21, 2026 20:11
@prathshenoy
prathshenoy force-pushed the feat/stovepipe-project-order-index branch from dde2b9a to f87bf65 Compare October 21, 2026 20:39
@prathshenoy
prathshenoy force-pushed the feat/stovepipe-project-order-index branch from f87bf65 to 6571781 Compare October 23, 2026 12:21
@prathshenoy prathshenoy changed the title feat(stovepipe): Add project ordering index feat(stovepipe): Add fact degree lookup table Oct 23, 2026
@prathshenoy
prathshenoy force-pushed the feat/stovepipe-project-order-index branch 4 times, most recently from b5fc4ca to 9603017 Compare October 23, 2026 13:07
@prathshenoy
prathshenoy force-pushed the feat/stovepipe-project-order-index branch from 9603017 to 4ddb37d Compare October 23, 2026 13:27
@prathshenoy
prathshenoy added this pull request to stack #926 October 23, 2026 13:27
@prathshenoy
prathshenoy marked this pull request as ready for review October 23, 2026 13:28
@prathshenoy
prathshenoy enabled stacked auto-merge October 23, 2026 13:51
@prathshenoy
prathshenoy added this pull request to the merge queue Oct 23, 2026
Merged via the queue into main with commit a67eae2 Oct 23, 2026
50 of 53 checks passed
@prathshenoy
prathshenoy deleted the feat/stovepipe-project-order-index branch October 23, 2026 15:07
DivyanshGarg380 pushed a commit to DivyanshGarg380/submitqueue that referenced this pull request Oct 24, 2026
## Summary
**What**:
- List newly indexed project results from most broken to least broken
through the degree-ordered fact lookup, with project-only continuation
tokens.
- Preserve caller-supplied order for targeted requests and detect
inconsistent indexed results.

**Why**:
- Bound listing work to the requested page without historical fallback
scans or sorting a non-key database attribute.

## Test Plan
- [x] Add unit tests.
- [x] Verify service API and browser pagination against real services.

## Revert Plan
- Revert this PR to restore project-ID listing. Keep index maintenance
and the additive table in place; authoritative facts are unchanged.

## Issue Links
-
[CODEM-608](https://linear.app/uber/issue/CODEM-608/order-by-most-broken)

## Stack

1. uber#908
1. uber#903
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