Repository navigation
Promote edge to stable: Delphi on Postgres behind a switch - #3009
Merged
Merged
Conversation
Lands on its own, as the routing guard requires, before the change that adds the reader. Claude-Session: https://claude.ai/code/session_018NVGBuYCk4EZmiz4csUv9s Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Lands on its own, as the routing guard requires, before the change that adds them. Claude-Session: https://claude.ai/code/session_018NVGBuYCk4EZmiz4csUv9s Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…witch (#3007) * Delphi on Postgres: queue-driven runs, run-bound results, readers behind a switch Delphi runs through the job queue as dependent per-step jobs; each step's result is stored in Postgres tied to its run. Readers can read from Postgres behind a switch that defaults to DynamoDB, so merging changes nothing until the switch is turned on. An importer command copies existing DynamoDB results; it is not run automatically. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018NVGBuYCk4EZmiz4csUv9s * Delphi on Postgres: writers follow the switch too With DELPHI_RESULT_BACKEND=postgres, job submission is admitted to the Postgres queue and narrative/statement writes go to Postgres, so nothing writes DynamoDB once the switch is on. The default stays DynamoDB. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018NVGBuYCk4EZmiz4csUv9s * Delphi on Postgres: fix lint, clippy and replay checks for the writers change Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018NVGBuYCk4EZmiz4csUv9s --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…ynamoDB) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018NVGBuYCk4EZmiz4csUv9s
Delphi Coverage Report
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ships #3006, #3007 and #3008.
Observable changes: none by default. DELPHI_RESULT_BACKEND defaults to DynamoDB; the Postgres queue path, Postgres readers and writers, and the importer command are present but inactive.
SCHEMA: this release applies three new migrations on deploy (through the migration runner, before old services stop): 000028_create_delphi_results, 000029_extend_delphi_graph_stages, 000030_create_delphi_writers. They create new tables for Delphi results and writers and extend the job-graph stage list; nothing existing is altered beyond that extension.
Rollback: redeploy the previous stable; the added tables are unused by it.
🤖 Generated with Claude Code
https://claude.ai/code/session_018NVGBuYCk4EZmiz4csUv9s