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

Enable indexscan on uncompressed part of partially compressed chunks #5589

Merged
merged 1 commit into from Apr 19, 2023

Conversation

konskov
Copy link
Contributor

@konskov konskov commented Apr 19, 2023

This was previously disabled as no data resided on the uncompressed chunk once it was compressed, but this is not the case anymore with partially compressed chunks, so we enable indexscan for the uncompressed chunk again.

Fixes #5432

Co-authored-by: Ante Kresic ante.kresic@gmail.com
(cherry picked from commit 5633960)

@konskov konskov marked this pull request as ready for review April 19, 2023 11:59
@github-actions
Copy link

@akuzm, @lkshminarayanan: please review this pull request.

Powered by pull-review

@codecov
Copy link

codecov bot commented Apr 19, 2023

Codecov Report

❗ No coverage uploaded for pull request base (2.10.x@e03aa0b). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 8738526 differs from pull request most recent head e46d44e. Consider uploading reports for the commit e46d44e to get more accurate results

@@            Coverage Diff            @@
##             2.10.x    #5589   +/-   ##
=========================================
  Coverage          ?   90.64%           
=========================================
  Files             ?      226           
  Lines             ?    52264           
  Branches          ?        0           
=========================================
  Hits              ?    47376           
  Misses            ?     4888           
  Partials          ?        0           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

This was previously disabled as no data resided on the
uncompressed chunk once it was compressed, but this is not
the case anymore with partially compressed chunks, so we
enable indexscan for the uncompressed chunk again.

Fixes timescale#5432

Co-authored-by: Ante Kresic <ante.kresic@gmail.com>
(cherry picked from commit 5633960)
@akuzm akuzm enabled auto-merge (rebase) April 19, 2023 13:13
@akuzm akuzm merged commit 4c71d76 into timescale:2.10.x Apr 19, 2023
47 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

2 participants