Clarify db index docs #13561
Merged
Clarify db index docs #13561
Conversation
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
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
saponifi3d
approved these changes
May 1, 2025
|
|
||
| When adding indexes to large tables you should use a `is_post_deployment` migration as creating the index could take longer than the migration statement timeout of 5s. | ||
|
|
||
| Note: These are created using `CONCURRENTLY`, so it's important to not set `atomic = True` for migrations that contain indexes. This is disabled by default. |
There was a problem hiding this comment.
I think it'd be a little clearer if the atomic = True was part of the first paragraph, like "these are things we should be doing when we want to generate a new index"
Then in the note, it can explain that it's because we use concurrently.
There was a problem hiding this comment.
Generally no action is required about atomic. It's false by default, so it's just a side note in case someone tries to enable it for some reason.
codyde
added a commit
that referenced
this pull request
May 1, 2025
* 'master' of https://github.com/getsentry/sentry-docs: Add "Data Collected" page for Godot SDK (and more) (#13533) Clarify FK docs (#13562) Clarify db index docs (#13561) Remove non-functioning video embed (#13557) Masinette integrations elba (#13503) Incorporate Uptime Monitors Into The Crons Management Page (#13507) feat(javascript): Add "Data Collected" (#13527) docs(relay): remove entries that are not scrubbed by default (#13555) flutter: Add note to app start integration (#13479) ref(flutter): move custom zone handling code snippet from init to troubleshooting (#13478) docs(python): add uv option to python install instructions (#13510) add docs for supabase integration (#13545) feat(native): add view hierarchy to develop docs (#13544) feat(autofix): Update autofix images (#13540) Ad alerts and dashboard page to new quickstart guide (#13522) Add data enrichment page to new onboarding guide (#13508) Add first onboarding guide to docs (#13462) (feat) Sentry MCP Documentation (#13524) feat(Capacitor): Migration guides for version 2 (#13022) docs(js): move API-related content from Manual Setup to APIs page (#13327)
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
No description provided.