Skip to content

feat: use the query-doctor postgres fork for CI runs#103

Open
Xetera wants to merge 8 commits intomainfrom
feat-use-our-fork
Open

feat: use the query-doctor postgres fork for CI runs#103
Xetera wants to merge 8 commits intomainfrom
feat-use-our-fork

Conversation

@Xetera
Copy link
Copy Markdown
Collaborator

@Xetera Xetera commented Mar 27, 2026

No description provided.

@Xetera Xetera force-pushed the feat-use-our-fork branch from 6dfb65f to e13653e Compare March 27, 2026 16:50
@Xetera Xetera force-pushed the feat-use-our-fork branch from 8634789 to e2372e0 Compare March 27, 2026 17:48
@Xetera Xetera force-pushed the feat-use-our-fork branch from 086c5df to cf14eb3 Compare March 27, 2026 18:08
@Xetera Xetera force-pushed the feat-use-our-fork branch from cf14eb3 to b38fb19 Compare March 27, 2026 18:10
@Xetera Xetera marked this pull request as ready for review March 27, 2026 18:23
@Xetera Xetera force-pushed the feat-use-our-fork branch from ffe87e3 to ca708e9 Compare March 27, 2026 18:27
Comment on lines +84 to +86
meanTime: 0,
calls: "1",
rows: "0",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

are we losing meanTime, calls, rows?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

These were all N/A fields to begin with no? Logs don't include that data

@Xetera Xetera force-pushed the feat-use-our-fork branch from ca708e9 to 93cc758 Compare March 27, 2026 18:36
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

85 queries analyzed

This PR improves queries

2 pre-existing issues

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

@Xetera Xetera force-pushed the feat-use-our-fork branch from 93cc758 to df0ee34 Compare March 27, 2026 18:41
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