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-645: pg_stat_monitor crashes PostgreSQL if there is citus .. #425

Merged
merged 1 commit into from
Nov 1, 2023

Conversation

codeforall
Copy link
Contributor

Do not look for the query in the hash if no query string is provided in the planner hook.

Do not look for the query in the hash if no query string is
provided in the planner hook.
Copy link

codecov bot commented Nov 1, 2023

Codecov Report

Merging #425 (823bfb9) into main (715a4ce) will decrease coverage by 0.32%.
Report is 1 commits behind head on main.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main     #425      +/-   ##
==========================================
- Coverage   91.02%   90.70%   -0.32%     
==========================================
  Files           3        3              
  Lines        1270     1270              
  Branches       34       35       +1     
==========================================
- Hits         1156     1152       -4     
- Misses         93       96       +3     
- Partials       21       22       +1     
Files Coverage Δ
pg_stat_monitor.c 90.00% <66.66%> (-0.36%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@codeforall codeforall requested a review from hqakhtar November 1, 2023 06:40
Copy link
Contributor

@hqakhtar hqakhtar left a comment

Choose a reason for hiding this comment

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

LGTM

@hqakhtar hqakhtar merged commit ddc5a17 into percona:main Nov 1, 2023
7 of 9 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.

2 participants