Skip to content

docs: update Supavisor application_name query#345

Closed
w3b6x9 wants to merge 1 commit intomainfrom
docs/application-name-query
Closed

docs: update Supavisor application_name query#345
w3b6x9 wants to merge 1 commit intomainfrom
docs/application-name-query

Conversation

@w3b6x9
Copy link
Copy Markdown
Member

@w3b6x9 w3b6x9 commented May 29, 2024

No description provided.

@w3b6x9 w3b6x9 requested review from abc3 and chasers May 29, 2024 01:36
Comment thread docs/migrating/pgbouncer.md Outdated
application_name
from pg_stat_activity
where application_name ilike '%Supavisor%'
where application_name = 'Supavisor'
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there's no point in doing ilike when we set it as Supavisor.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is also a Supavisor auth_query one

@w3b6x9 w3b6x9 force-pushed the docs/application-name-query branch 2 times, most recently from bccf097 to 857b4c9 Compare May 29, 2024 01:45
@w3b6x9 w3b6x9 force-pushed the docs/application-name-query branch from 857b4c9 to 4c56def Compare May 29, 2024 02:45
application_name
from pg_stat_activity
where application_name ilike '%Supavisor%'
where application_name in ('Supavisor', 'Supavisor auth_query')
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chasers including the auth query one.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But why? Now we have to remember to add any here if we update them.

Copy link
Copy Markdown
Contributor

@abc3 abc3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

conflicts with #343

@hauleth hauleth requested a review from a team as a code owner January 14, 2025 19:07
@hauleth hauleth closed this Mar 5, 2025
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.

4 participants