Skip to content

perf: add external_id index on users table#870

Merged
v0idpwn merged 2 commits intomainfrom
perf/add-external-id-users
Mar 7, 2026
Merged

perf: add external_id index on users table#870
v0idpwn merged 2 commits intomainfrom
perf/add-external-id-users

Conversation

@v0idpwn
Copy link
Copy Markdown
Member

@v0idpwn v0idpwn commented Mar 7, 2026

No description provided.

@v0idpwn v0idpwn requested a review from a team as a code owner March 7, 2026 13:02
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 7, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 6ea988fa-bd17-4b01-99f5-6140aa2836cf

📥 Commits

Reviewing files that changed from the base of the PR and between be386e7 and 91f6563.

📒 Files selected for processing (1)
  • priv/repo/migrations/20260307125522_add_users_tenant_external_id_index.exs

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Database optimization: Added indexing to enhance query performance and improve system efficiency.

Walkthrough

A new Ecto migration module is introduced that creates a concurrent index on the tenant_external_id column of the users table in the _supavisor schema. The migration uses create_if_not_exists to ensure idempotency, disables default DDL transaction handling, and executes with the concurrently: true option to avoid locking during index creation.

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@v0idpwn v0idpwn merged commit 85acdde into main Mar 7, 2026
13 of 15 checks passed
@v0idpwn v0idpwn deleted the perf/add-external-id-users branch March 7, 2026 22:03
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.

2 participants