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

Set name for COPY insert buffer hash table #5350

Merged
merged 1 commit into from Feb 20, 2023

Conversation

svenklemm
Copy link
Member

@svenklemm svenklemm commented Feb 20, 2023

Having the hash table named makes debugging easier as the name is used for the MemoryContext used by the hash table.

Disable-check: force-changelog-changed

Having the hash table named makes debugging easier as the name
is used for the MemoryContext used by the hash table.
@github-actions
Copy link

@shhnwz, @RafiaSabih: please review this pull request.

Powered by pull-review

@svenklemm svenklemm enabled auto-merge (rebase) February 20, 2023 16:14
@codecov
Copy link

codecov bot commented Feb 20, 2023

Codecov Report

Merging #5350 (f0af830) into main (7e43c70) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #5350      +/-   ##
==========================================
+ Coverage   90.70%   90.80%   +0.10%     
==========================================
  Files         225      225              
  Lines       52020    45578    -6442     
==========================================
- Hits        47183    41386    -5797     
+ Misses       4837     4192     -645     
Impacted Files Coverage Δ
src/copy.c 94.55% <100.00%> (-0.14%) ⬇️
tsl/src/remote/tuplefactory.c 72.60% <0.00%> (-16.91%) ⬇️
src/bgw_policy/chunk_stats.c 81.53% <0.00%> (-4.01%) ⬇️
tsl/test/src/remote/dist_commands.c 90.62% <0.00%> (-3.97%) ⬇️
src/nodes/chunk_append/transform.c 96.66% <0.00%> (-3.34%) ⬇️
src/partitioning.c 85.98% <0.00%> (-3.10%) ⬇️
src/loader/bgw_launcher.c 89.22% <0.00%> (-2.85%) ⬇️
src/import/planner.c 62.40% <0.00%> (-2.76%) ⬇️
src/bgw/scheduler.c 84.22% <0.00%> (-2.74%) ⬇️
tsl/src/fdw/estimate.c 77.18% <0.00%> (-2.36%) ⬇️
... and 193 more

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

@svenklemm svenklemm merged commit 0976634 into timescale:main Feb 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants