-
Notifications
You must be signed in to change notification settings - Fork 67
Update postgres12_end_of_life_notice.mdx #1027
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM aside from line 54
SELECT * FROM pg_stat_progress_create_index; | ||
``` | ||
Look for the `blocks_done` value in the output. Run the query at intervals—if this number increases, reindexing is ongoing. For better readability, enable extended output with `\x` command. | ||
Also note that the reindexing process might take a bit of time reindexing the rockskip table if the reindexing is happening in the code intel DB. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line 54/55 doesn't make sense. Could you check what you were trying to communicate here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@airamare01 The reindexing process applies to all three databases. In Atlassian's case, it specifically occurred in the code intel DB, where we observed that the rockskip table took significantly longer to reindex. That's why I included the caveat.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@airamare01 @mibali pretty old PR but is this change still needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deferring to @mibali on this PR.
This PR Introduced guidance on using
pg_stat_progress_create_index
to track reindexing status, helping admins confirm whether the process is progressing.Pull Request approval
You will need to get your PR approved by at least one member of the Sourcegraph team. For reviews of docs formatting, styles, and component usage, please tag the docs team via the #docs Slack channel.