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

Fix dist_hypertable test #5281

Merged
merged 1 commit into from Feb 3, 2023

Conversation

erimatnor
Copy link
Contributor

@erimatnor erimatnor commented Feb 3, 2023

A previous change accidentally broke the dist_hypertable test so that it prematurely exited. This change restores the test so that it executes properly.

Disable-check: force-changelog-changed

A previous change accidentally broke the dist_hypertable test so that
it prematurely exited. This change restores the test so that it
executes properly.
@erimatnor erimatnor added testing multinode tech-debt Needs refactoring and improvement tasks related to the source code and its architecture. labels Feb 3, 2023
@erimatnor erimatnor self-assigned this Feb 3, 2023
@codecov
Copy link

codecov bot commented Feb 3, 2023

Codecov Report

Merging #5281 (9c65ea6) into main (c4d8f35) will increase coverage by 0.28%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #5281      +/-   ##
==========================================
+ Coverage   89.03%   89.32%   +0.28%     
==========================================
  Files         225      225              
  Lines       51843    51842       -1     
==========================================
+ Hits        46157    46306     +149     
+ Misses       5686     5536     -150     
Impacted Files Coverage Δ
src/loader/bgw_message_queue.c 86.36% <0.00%> (-2.85%) ⬇️
src/bgw/scheduler.c 83.88% <0.00%> (-1.54%) ⬇️
src/bgw/job_stat.c 91.87% <0.00%> (-0.32%) ⬇️
tsl/src/reorder.c 85.49% <0.00%> (-0.22%) ⬇️
tsl/src/bgw_policy/job.c 87.45% <0.00%> (-0.05%) ⬇️
src/process_utility.c 90.17% <0.00%> (+0.04%) ⬆️
src/chunk.c 92.08% <0.00%> (+0.05%) ⬆️
src/hypertable.c 87.43% <0.00%> (+0.16%) ⬆️
tsl/src/data_node.c 96.08% <0.00%> (+0.23%) ⬆️
src/ts_catalog/tablespace.c 95.20% <0.00%> (+0.29%) ⬆️
... and 18 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 c4d8f35...9c65ea6. Read the comment docs.

@erimatnor erimatnor marked this pull request as ready for review February 3, 2023 12:15
@erimatnor erimatnor merged commit 206056c into timescale:main Feb 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
multinode tech-debt Needs refactoring and improvement tasks related to the source code and its architecture. testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants