Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Changed lock order for MERGE of range partitions + change for issue #247 #248

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

Merged
kovdb75 merged 2 commits into master from PGPRO-6538
May 18, 2022

Conversation

Copy link
Collaborator

@kovdb75 kovdb75 commented May 16, 2022

  1. Changed lock order for MERGE of range partitions: the parent table is locked first and then are locked the partitions.
  2. Change for issue relation "5_3" does not exist error when using pg_pathman with Citus #247 : need to skip non-existing relations for Citus compatibility.

Copy link

codecov bot commented May 16, 2022
edited
Loading

Codecov Report

Merging #248 (8101d6d) into master (e57e04d) will increase coverage by 0.59%.
The diff coverage is 100.00%.

❗ Current head 8101d6d differs from pull request most recent head 66543e7. Consider uploading reports for the commit 66543e7 to get more accurate results

@@ Coverage Diff @@
## master #248 +/- ##
==========================================
+ Coverage 91.45% 92.04% +0.59% 
==========================================
 Files 43 34 -9 
 Lines 7255 6626 -629 
==========================================
- Hits 6635 6099 -536 
+ Misses 620 527 -93 
Impacted Files Coverage Δ
src/pl_range_funcs.c 96.48% <100.00%> (-0.02%) ⬇️
src/utility_stmt_hooking.c 92.77% <100.00%> (+0.55%) ⬆️
src/planner_tree_modification.c 86.74% <0.00%> (-1.79%) ⬇️
src/include/compat/pg_compat.h 92.30% <0.00%> (-1.03%) ⬇️
src/utils.c 92.10% <0.00%> (-0.66%) ⬇️
src/init.c 91.02% <0.00%> (-0.22%) ⬇️
src/runtime_merge_append.c 69.51% <0.00%> (-0.18%) ⬇️
src/partition_filter.c 96.44% <0.00%> (-0.14%) ⬇️
src/hooks.c 92.28% <0.00%> (-0.11%) ⬇️
src/partition_creation.c 96.81% <0.00%> (-0.09%) ⬇️
... and 24 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e57e04d...66543e7. Read the comment docs.

@kovdb75 kovdb75 merged commit eec4e1d into master May 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

1 participant

AltStyle によって変換されたページ (->オリジナル) /