Skip to content

Drop: Unused chunks_metadata table#123

Merged
OpsBotPrime merged 2 commits into
masterfrom
reinier/drop-chunks-metadata
Jul 10, 2026
Merged

Drop: Unused chunks_metadata table#123
OpsBotPrime merged 2 commits into
masterfrom
reinier/drop-chunks-metadata

Conversation

@ReinierMaas

Copy link
Copy Markdown
Contributor

As discussed offline with @jerbaroo & @Qqwy:

chunks_metadata doesn't contain any additional data compared to looking at the submissions_metadata and can be safely dropped from the database.

@ReinierMaas ReinierMaas requested a review from jerbaroo July 10, 2026 11:57
@ReinierMaas ReinierMaas self-assigned this Jul 10, 2026
@@ -0,0 +1 @@
DROP TABLE chunks_metadata;

@Qqwy Qqwy Jul 10, 2026

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.

Nit: It would be nice to make the migration bidirectional, just like we've been doing for most others.

(While double-checking this nit I saw that we did not do this correctly in the last two migrations either. I'll open an issue for that)

EDIT: #124

@ReinierMaas

Copy link
Copy Markdown
Contributor Author

@OpsBotPrime merge on Friday!

Either me or @jerbaroo can pickup #124.

@OpsBotPrime

Copy link
Copy Markdown
Contributor

Pull request approved for merge by @ReinierMaas, waiting for rebase behind one pull request.

@OpsBotPrime

Copy link
Copy Markdown
Contributor

Speculatively rebased as 08b8288 behind 1 other PR, waiting for CI …

@OpsBotPrime

Copy link
Copy Markdown
Contributor

CI job 🟡 started.

ReinierMaas and others added 2 commits July 10, 2026 17:17
Approved-by: ReinierMaas
Priority: Normal
Auto-deploy: false
@OpsBotPrime

Copy link
Copy Markdown
Contributor

Rebased as 5935222, waiting for CI …

@OpsBotPrime

Copy link
Copy Markdown
Contributor

CI job 🟡 started.

@OpsBotPrime OpsBotPrime force-pushed the reinier/drop-chunks-metadata branch from 86e1274 to 5935222 Compare July 10, 2026 15:27
@OpsBotPrime OpsBotPrime merged commit 5935222 into master Jul 10, 2026
6 of 7 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.

3 participants