Skip to content

Split query log capture into a log source and a log type - #139

Merged
teknogeek0 merged 1 commit into
mainfrom
sync/workload-log-config
Oct 9, 2026
Merged

teknogeek0 merged 1 commit into
mainfrom
sync/workload-log-config

Conversation

@teknogeek0

Copy link
Copy Markdown
Collaborator

Replaces the combined capture_log_source values with capture_log_type (statement or pgaudit) and capture_log_source (file or log_fdw). Old source values are rejected at load, so existing workload configs that set capture_log must change. Ported from the dev repo.

Tests: 1516 passed, 11 skipped. flake8, black and mypy pass.

capture_log_type (statement or pgaudit) is required when capture_log is
on. capture_log_source is now file or log_fdw. The tool reads whether
an exported file is plain text, csvlog or a pgAudit JSON export. The old
source values pgaudit, pgaudit-json, stderr and auto are rejected at
load, and the error names the settings that replace them.

@robdaly93 robdaly93 left a comment

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.

LGTM

@teknogeek0
teknogeek0 merged commit a0354dd into main Oct 9, 2026
7 checks passed
@teknogeek0
teknogeek0 deleted the sync/workload-log-config branch October 9, 2026 19:50
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