Skip to content

Conversation

svenklemm
Copy link
Member

  • Get rid of unused ChunkDispatch fields
  • Move has_dropped_attrs to ChunkTupleRouting
  • ++
  • Remove ChunkInsertState from ChunkDispatch
  • Remove ChunkDispatch from ModifyHypertable

Copy link

@Poroma-Banerjee, @pnthao: please review this pull request.

Powered by pull-review

Copy link

codecov bot commented Sep 11, 2025

Codecov Report

❌ Patch coverage is 85.10638% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.51%. Comparing base (c6d8af6) to head (61f9507).

Files with missing lines Patch % Lines
src/nodes/modify_hypertable.c 81.81% 2 Missing and 2 partials ⚠️
src/nodes/modify_hypertable_exec.c 86.95% 1 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8619      +/-   ##
==========================================
+ Coverage   82.46%   82.51%   +0.05%     
==========================================
  Files         248      248              
  Lines       47649    47605      -44     
  Branches    12102    12095       -7     
==========================================
- Hits        39293    39282      -11     
- Misses       3489     3510      +21     
+ Partials     4867     4813      -54     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@svenklemm svenklemm changed the title sven/rip dispatch Remove ChunkDispatch from ModifyHypertable Sep 11, 2025
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.

1 participant