Skip to content

Refactor code to reduce reliance on compressed_chunk_id - #9966

Merged
svenklemm merged 1 commit into
mainfrom
sven/compressed_chunk_id3
Jun 8, 2026
Merged

Refactor code to reduce reliance on compressed_chunk_id#9966
svenklemm merged 1 commit into
mainfrom
sven/compressed_chunk_id3

Conversation

@svenklemm

@svenklemm svenklemm commented Jun 5, 2026

Copy link
Copy Markdown
Member

The source of truth for chunk <-> compressed chunk link is
_timescaledb_catalog.compression_settings. This patch adjusts
the code to use that instead of compressed_chunk_id.

Disable-check: force-changelog-file

@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown

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

Powered by pull-review

@codecov

codecov Bot commented Jun 5, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 94.28571% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/chunk.c 92.30% 0 Missing and 1 partial ⚠️
src/process_utility.c 66.66% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@svenklemm
svenklemm force-pushed the sven/compressed_chunk_id3 branch 4 times, most recently from 90f3ab0 to bc150c5 Compare June 7, 2026 16:31
The source of truth for chunk <-> compressed chunk link is
_timescaledb_catalog.compression_settings. This patch adjusts
the code to use that instead of compressed_chunk_id.
@svenklemm
svenklemm force-pushed the sven/compressed_chunk_id3 branch from bc150c5 to 4aa3bc9 Compare June 7, 2026 17:00
@svenklemm
svenklemm requested review from antekresic and Copilot and removed request for Copilot June 8, 2026 07:06
@svenklemm
svenklemm merged commit f3d27a8 into main Jun 8, 2026
64 of 69 checks passed
@svenklemm
svenklemm deleted the sven/compressed_chunk_id3 branch June 8, 2026 11:38
@timescale-automation timescale-automation added the released-2.28.0 Released in 2.28.0 label Jun 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released-2.28.0 Released in 2.28.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants