Skip to content

chore(api): bump etl to rune-count limits + slug-collision index (#340)#919

Merged
raymondjacobson merged 1 commit into
mainfrom
chore/bump-etl-rune-count
Jun 8, 2026
Merged

chore(api): bump etl to rune-count limits + slug-collision index (#340)#919
raymondjacobson merged 1 commit into
mainfrom
chore/bump-etl-rune-count

Conversation

@raymondjacobson

@raymondjacobson raymondjacobson commented Jun 8, 2026

Copy link
Copy Markdown
Member

Summary

Bumps github.com/OpenAudio/go-openaudio and .../pkg/etl from 5d2e19a (#901) to 7062cd9. Because 7062cd9 is downstream of both fixes, this single bump ships two changes to core-indexer:

Supersedes #912 (the standalone slug bump). #912 can be closed — this bump includes everything in it plus #340.

Changes

  • go.mod / go.sum: both go-openaudio modules → v1.3.1-0.20260608175930-7062cd90dff5

⚠️ Deploy note (carried from #337) — pre-build the indexes CONCURRENTLY

This bump pulls migration 0031, which builds two route indexes non-concurrently (single-transaction migration runner), taking an ACCESS EXCLUSIVE lock on track_routes (~2M rows / 493 MB) on indexer boot. To avoid stalling route writes on deploy, pre-build them in prod first — IF NOT EXISTS then makes the migration a no-op:

CREATE INDEX CONCURRENTLY IF NOT EXISTS track_routes_owner_title_slug_idx
  ON track_routes (owner_id, title_slug, collision_id);
CREATE INDEX CONCURRENTLY IF NOT EXISTS playlist_routes_owner_title_slug_idx
  ON playlist_routes (owner_id, title_slug, collision_id);

(The rune-count change in #340 is pure validation logic — no migration, no special rollout.)

Test plan

  • go mod tidy — only go.mod/go.sum changed
  • go build ./... passes
  • resolved module contains the rune-count fix (RuneCountInString in validate.go) and migration 0031
  • Post-deploy: g1ld3dd (and other multi-byte-name users) can save profile updates / artist picks; 0031 applied or no-op; Track/Playlist max processing times drop

🤖 Generated with Claude Code

Bumps go-openaudio to 7062cd9, which is downstream of both:
- #340 fix(etl): count runes not bytes in name/bio/handle/description
  limits -- unblocks profiles with multi-byte names (e.g. emoji) that
  could not save updates, including artist picks.
- #337 perf(etl): index track_routes/playlist_routes on
  (owner_id, title_slug, collision_id) -- migration 0031.

Supersedes the standalone slug-collision bump (#912): this single bump
ships both changes.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@raymondjacobson raymondjacobson merged commit 01f3159 into main Jun 8, 2026
5 checks passed
@raymondjacobson raymondjacobson deleted the chore/bump-etl-rune-count branch June 8, 2026 18:16
raymondjacobson added a commit that referenced this pull request Jun 8, 2026
## Summary

Bumps `github.com/OpenAudio/go-openaudio` and `.../pkg/etl` from
`7062cd9` (#919) to **`4cccb46`** to consume go-openaudio **#341**.

That change fixes the User Create/Update handlers silently dropping
`instagram_handle`, `twitter_handle`, `tiktok_handle`, `website`, and
`donation` — so users editing their profile could not add or change
social links. Reported by `audius.co/allatonce` and
`audius.co/peaceluis`. Production data: 167k+ users have an *unverified*
Instagram handle, confirming these are normal profile fields, not
verification-only.

`core-indexer` runs this vendored `audius/api` image, so this bump is
required to ship the fix.

## Changes
- `go.mod` / `go.sum`: both go-openaudio modules →
`v1.3.1-0.20260608204715-4cccb46f1332`

## Deploy note — migration 0032 (low risk)
This bump pulls ETL migration `0032`, which adds the `website` +
`donation` columns with `IF NOT EXISTS`. In prod these columns already
exist (inherited legacy schema), so 0032 is a **no-op there** — no lock
concern (unlike the 0031 index build). No special rollout needed.

## Test plan
- [x] `go mod tidy` — only go.mod/go.sum changed
- [x] `go build ./...` passes
- [x] resolved module contains the social-field writes
(`instagram_handle` in user_update.go) and migration `0032`
- [ ] Post-deploy: `allatonce` / `peaceluis` can save IG handle +
website; `verified_with_*` unchanged

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
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