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

Restrict creation of partial cagg #6056

Closed
wants to merge 1 commit into from
Closed

Restrict creation of partial cagg #6056

wants to merge 1 commit into from

Conversation

pdipesh02
Copy link
Contributor

Block creation of old format cagg. However, partial aggregates can be created under a custom boolean variable block_old_format_cagg. This variable can be used only for development purposes like validating migration from old format to new format cagg until we completely remove the old format cagg.

@codecov
Copy link

codecov bot commented Sep 8, 2023

Codecov Report

Merging #6056 (e5d4438) into main (ef783c4) will decrease coverage by 7.79%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##             main    #6056      +/-   ##
==========================================
- Coverage   81.43%   73.64%   -7.79%     
==========================================
  Files         243      243              
  Lines       55948    49083    -6865     
  Branches    12389    12285     -104     
==========================================
- Hits        45562    36148    -9414     
+ Misses       8048     7197     -851     
- Partials     2338     5738    +3400     
Files Changed Coverage Δ
tsl/src/continuous_aggs/repair.c 66.66% <75.00%> (-16.99%) ⬇️
src/guc.c 94.04% <100.00%> (-2.59%) ⬇️
tsl/src/continuous_aggs/common.c 79.04% <100.00%> (-11.02%) ⬇️

... and 220 files with indirect coverage changes

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

@pdipesh02 pdipesh02 force-pushed the ts_old_cagg_2 branch 2 times, most recently from 370648c to 1935d26 Compare September 10, 2023 15:45
Block creation of old format cagg. However, partial aggregates can be
created under a custom boolean variable `block_old_format_cagg`. This
variable can be used only for development purposes like validating
migration from old format to new format cagg until we completely remove
the old format cagg.
@horzsolt
Copy link

horzsolt commented Oct 3, 2023

On other PRs we will continue this effort.

@horzsolt horzsolt closed this Oct 3, 2023
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.

3 participants