Skip to content
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

pg_replication_slots in standby #147

Merged
merged 5 commits into from
Jan 10, 2024
Merged

pg_replication_slots in standby #147

merged 5 commits into from
Jan 10, 2024

Conversation

fmbiete
Copy link

@fmbiete fmbiete commented Aug 30, 2023

WAL functions cannot be executed in standby servers

ERROR:  recovery is in progress
HINT:  WAL control functions cannot be executed during recovery.

In PostgreSQL 16 replication slots persist in the standby servers. This is also the case when using extensions like pg_failover_slots that transfer the slot information to the standby.

This condition will make those queries to return values only in the primary node avoiding the errors.

WAL functions cannot be executed in standby servers
```
ERROR:  recovery is in progress
HINT:  WAL control functions cannot be executed during recovery.
```

In PostgreSQL 16 replication slots persist in the standby servers.
This is also the case when using extensions like pg_failover_slots that transfer the slot information to the standby.

This condition will make those queries to return values only in the primary node avoiding the errors.
@fmbiete fmbiete requested a review from a team as a code owner August 30, 2023 09:22
@it-percona-cla
Copy link

it-percona-cla commented Aug 30, 2023

CLA assistant check
All committers have signed the CLA.

@BupycHuk
Copy link
Member

BupycHuk commented Sep 1, 2023

Hello @fmbiete, thank you for your contribution. We recommend creating the same PR for the upstream project.

@fmbiete
Copy link
Author

fmbiete commented Sep 1, 2023

prometheus-community#899

But since PMM is including Percona forked postgres_exporter, can we merge the change here?

@JiriCtvrtka JiriCtvrtka temporarily deployed to CI October 17, 2023 07:06 — with GitHub Actions Inactive
@JiriCtvrtka JiriCtvrtka temporarily deployed to CI October 17, 2023 07:06 — with GitHub Actions Inactive
@JiriCtvrtka JiriCtvrtka temporarily deployed to CI October 17, 2023 07:06 — with GitHub Actions Inactive
@JiriCtvrtka JiriCtvrtka temporarily deployed to CI October 17, 2023 07:06 — with GitHub Actions Inactive
@JiriCtvrtka JiriCtvrtka temporarily deployed to CI October 17, 2023 07:06 — with GitHub Actions Inactive
@BupycHuk BupycHuk temporarily deployed to CI October 26, 2023 17:51 — with GitHub Actions Inactive
@BupycHuk BupycHuk temporarily deployed to CI October 26, 2023 17:51 — with GitHub Actions Inactive
@BupycHuk BupycHuk temporarily deployed to CI October 26, 2023 17:51 — with GitHub Actions Inactive
@BupycHuk BupycHuk temporarily deployed to CI October 26, 2023 17:51 — with GitHub Actions Inactive
@BupycHuk BupycHuk temporarily deployed to CI October 26, 2023 17:51 — with GitHub Actions Inactive
@JiriCtvrtka
Copy link

@fmbiete Thanks for your contribution.

@JiriCtvrtka JiriCtvrtka merged commit 69eb75f into percona:main Jan 10, 2024
6 checks passed
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