timescaledb icon indicating copy to clipboard operation
timescaledb copied to clipboard

Fix continuous aggregate migrate check constraint

Open fabriziomello opened this issue 1 year ago • 1 comments

Instances upgraded to 2.8.0 will end up with a wrong check constraint in catalog table continuous_aggregate_migrate_plan_step.

Fixed it by removing and adding the constraint with the correct checks.

Fix #4727

fabriziomello avatar Sep 21 '22 13:09 fabriziomello

Codecov Report

Merging #4739 (524bcb0) into main (5600fc0) will decrease coverage by 0.03%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4739      +/-   ##
==========================================
- Coverage   90.92%   90.88%   -0.04%     
==========================================
  Files         224      224              
  Lines       42406    42407       +1     
==========================================
- Hits        38556    38541      -15     
- Misses       3850     3866      +16     
Impacted Files Coverage Δ
tsl/src/compression/api.c 95.70% <100.00%> (ø)
tsl/src/compression/compression.c 95.26% <100.00%> (+0.01%) :arrow_up:
src/bgw/scheduler.c 81.57% <0.00%> (-3.22%) :arrow_down:
src/loader/bgw_message_queue.c 85.52% <0.00%> (-2.64%) :arrow_down:
src/bgw/job.c 94.41% <0.00%> (-0.24%) :arrow_down:
tsl/src/bgw_policy/job.c 88.11% <0.00%> (-0.05%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 4cb3835...524bcb0. Read the comment docs.

codecov[bot] avatar Sep 21 '22 13:09 codecov[bot]

What about upgrade/downgrade tests?

mfundul avatar Sep 22 '22 07:09 mfundul

What about upgrade/downgrade tests?

I don't know yet what happened, but in any case it will be material for another PR.

fabriziomello avatar Sep 22 '22 13:09 fabriziomello