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

Disable autovacuum on compressed hypercore chunk #7358

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mkindahl
Copy link
Contributor

@mkindahl mkindahl commented Oct 17, 2024

Autovacuum will run on all relations that are not explicitly disabled,
which can lead to unnecessarily vacuuming the compressed relation
twice: once as part of the hypercore table access method and once as a
separate relation.

This commit disables autovacuum_enabled for the compressed chunks
when the main chunk is turned into a hypercore chunk. It deals with
both the cases that you have a partially compressed chunk.

Disable-check: commit-count

@mkindahl mkindahl force-pushed the hypercore-disable-compressed-vacuum branch from 401a8b5 to eeabdec Compare October 17, 2024 13:34
Copy link

codecov bot commented Oct 17, 2024

Codecov Report

Attention: Patch coverage is 79.16667% with 10 lines in your changes missing coverage. Please review.

Project coverage is 82.59%. Comparing base (59f50f2) to head (df61898).
Report is 547 commits behind head on main.

Files with missing lines Patch % Lines
src/utils.c 69.69% 3 Missing and 7 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7358      +/-   ##
==========================================
+ Coverage   80.06%   82.59%   +2.53%     
==========================================
  Files         190      229      +39     
  Lines       37181    42709    +5528     
  Branches     9450    10725    +1275     
==========================================
+ Hits        29770    35277    +5507     
- Misses       2997     3156     +159     
+ Partials     4414     4276     -138     

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

@mkindahl mkindahl force-pushed the hypercore-disable-compressed-vacuum branch 2 times, most recently from d898b7a to b8ade07 Compare October 17, 2024 14:23
Autovacuum will run on all relations that are not explicitly disabled,
which can lead to unnecessarily vacuuming the compressed relation
twice: once as part of the hypercore table access method and once as a
separate relation.

This commit disables `autovacuum_enabled` for the compressed chunks
when the main chunk is turned into a hypercore chunk. It deals with
both the cases that you have a partially compressed chunk.
@mkindahl mkindahl force-pushed the hypercore-disable-compressed-vacuum branch from b8ade07 to df61898 Compare October 17, 2024 18:38
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