Skip to content

feat: allow pg_stat_statements as query source in ci#138

Open
Xetera wants to merge 2 commits into
mainfrom
feat-unify-logs-pss
Open

feat: allow pg_stat_statements as query source in ci#138
Xetera wants to merge 2 commits into
mainfrom
feat-unify-logs-pss

Conversation

@Xetera
Copy link
Copy Markdown
Collaborator

@Xetera Xetera commented May 20, 2026

Previously we were forcing users to supply a LOG_PATH which is not really convenient. This puts things more in-line with the usual execution by introducing a RecentQuerySource interface classes can implement to provide an array of queries. With pgbadger that's by reading logs. With the regular PgConnector that's by querying PSS.

This change makes LOG_PATH optional.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Query Doctor Analysis

View full run details

2 queries analyzed

2 pre-existing issues

Using assumed statistics (10000000 rows/table). For better results, sync production stats.

@Xetera Xetera force-pushed the feat-unify-logs-pss branch from f1a34eb to 12cb186 Compare May 20, 2026 19:51
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.

1 participant