Skip to content

chore(weave): add scorer_* columns to feedback table (031)#6906

Draft
nikumar1206 wants to merge 1 commit into
masterfrom
nikhil/scorer-feedback-columns-migration
Draft

chore(weave): add scorer_* columns to feedback table (031)#6906
nikumar1206 wants to merge 1 commit into
masterfrom
nikhil/scorer-feedback-columns-migration

Conversation

@nikumar1206
Copy link
Copy Markdown
Contributor

@nikumar1206 nikumar1206 commented May 20, 2026

Description

Adds four typed columns to the `feedback` table:

  • `scorer_tags` `Array(String)`
  • `scorer_ratings` `Map(LowCardinality(String), Float64)`
  • `scorer_reasons` `Map(String, String)`
  • `scorer_confidences` `Map(String, Float64)`

Non-null with empty defaults so existing rows need no backfill. DDL only — no readers or writers touch the columns yet.

Testing

Applied locally with `make migrate_local`; up + down both verified.

Copy link
Copy Markdown
Contributor Author

nikumar1206 commented May 20, 2026

@codecov
Copy link
Copy Markdown

codecov Bot commented May 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@nikumar1206 nikumar1206 force-pushed the nikhil/scorer-feedback-columns-migration branch 2 times, most recently from 7d85f02 to cf341ff Compare May 20, 2026 19:17
@nikumar1206 nikumar1206 force-pushed the nikhil/scorer-feedback-columns-migration branch from cf341ff to 34b26b1 Compare May 20, 2026 23:47
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.

1 participant