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

Backport to 2.14.x: #6774: Fully recompress unordered compressed chunks #6783

Closed
wants to merge 1 commit into from

Conversation

timescale-automation
Copy link

This is an automated backport of #6774: Fully recompress unordered compressed chunks.
This PR will be merged automatically after all the relevant CI checks pass. If this fix should not be backported, or will be backported manually, just close this PR. You can use the backport branch to add your changes, it won't be modified automatically anymore.

For more details, please see the documentation

Original description

Fully recompress unordered compressed chunks

Recent changes switched recompression of unordered compressed chunks to use segmentwise recompression which is designed to work with partial chunks only. This change reverts that back to full recompression.

Disable-check: force-changelog-file

Recent changes switched recompression of unordered compressed
chunks to use segmentwise recompression which is designed to work
with partial chunks only. This change reverts that back to full
recompression.

(cherry picked from commit 24552be)
@timescale-automation timescale-automation added the is-auto-backport PR created by backport automation label Mar 25, 2024
@timescale-automation timescale-automation enabled auto-merge (rebase) March 25, 2024 08:54
Copy link

codecov bot commented Mar 25, 2024

Codecov Report

Attention: Patch coverage is 33.33333% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 79.63%. Comparing base (39616e8) to head (ddd1942).
Report is 14 commits behind head on 2.14.x.

Files Patch % Lines
tsl/src/compression/api.c 33.33% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           2.14.x    #6783      +/-   ##
==========================================
- Coverage   79.73%   79.63%   -0.11%     
==========================================
  Files         190      190              
  Lines       37343    37195     -148     
  Branches     9483     9439      -44     
==========================================
- Hits        29777    29620     -157     
- Misses       3147     3179      +32     
+ Partials     4419     4396      -23     

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

@antekresic
Copy link
Contributor

Released with 2.15, closing.

@antekresic antekresic closed this May 13, 2024
auto-merge was automatically disabled May 13, 2024 08:24

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is-auto-backport PR created by backport automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants