Repository navigation
feat: add timestamp to agent index table - #129
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The blocking index creation could stall production writes while migrating a large agent-index table.
Review effort: Balanced
Findings: 1
Open (2)
What changed in this PR
Adds timestamps to PostgreSQL agent-index entries to support age-based purging.
Changes:
- Adds
inserted_atwith backfill and an index. - Tests migration behavior for existing and new rows.
| File | Description |
|---|---|
internal/migrations/sql/00006_agent_index_inserted_at.sql |
Adds the timestamp column and purge index. |
internal/migrations/agent_index_test.go |
Verifies backfill, defaults, and index creation. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
fil-forge-bot Bot
added a commit
to fil-forge/infra-central
that referenced
this pull request
Oct 5, 2026
Published from fil-forge/sprue#129 - Digest: `sha256:79eb33d967f163fb7370ce4cacb757428f551466625e191ed774efa949327cb1` - Commit: fil-forge/sprue@9b40689 - Publish run: https://github.com/fil-forge/sprue/actions/runs/37341126537 Merging applies [`terraform/envs/dev/apps`](https://github.com/fil-forge/infra-central/tree/main/terraform/envs/dev/apps) with no further confirmation. Co-authored-by: fil-forge-bot[bot] <318653112+fil-forge-bot[bot]@users.noreply.github.com>
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
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.
refs https://linear.app/filecoin-foundation/issue/FIL-1300