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

Remove remote connection handling code #6432

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

svenklemm
Copy link
Member

@svenklemm svenklemm commented Dec 15, 2023

Disable-check: force-changelog-file

Copy link

@antekresic, @mahipv: please review this pull request.

Powered by pull-review

@svenklemm svenklemm force-pushed the multinode8 branch 3 times, most recently from 7b3c5c0 to a5258fb Compare December 15, 2023 16:58
Copy link

codecov bot commented Dec 15, 2023

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (3d8ec1e) 75.65% compared to head (84f462d) 86.38%.

Files Patch % Lines
src/process_utility.c 66.66% 2 Missing ⚠️
tsl/src/bgw_policy/job.c 50.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #6432       +/-   ##
===========================================
+ Coverage   75.65%   86.38%   +10.73%     
===========================================
  Files         212      187       -25     
  Lines       48796    42331     -6465     
  Branches    10868     9333     -1535     
===========================================
- Hits        36915    36569      -346     
+ Misses      10125     4077     -6048     
+ Partials     1756     1685       -71     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@svenklemm svenklemm force-pushed the multinode8 branch 3 times, most recently from 4da69bc to 2f6aa8c Compare December 15, 2023 17:38
Remove the code used by multinode to handle remote connections.
This patch completely removes tsl/src/remote and any remaining
distributed hypertable checks.
@svenklemm svenklemm changed the title Remove mutlinode cache invalidation Remove remote connection handling code Dec 15, 2023
@svenklemm svenklemm enabled auto-merge (rebase) December 15, 2023 17:47
@svenklemm svenklemm merged commit 6395b24 into timescale:main Dec 15, 2023
64 of 65 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.

None yet

3 participants