search

timescale / timescaledb

An open-source time-series SQL database optimized for fast ingest and complex queries. Packaged as a PostgreSQL extension.
https://www.timescale.com/
Other
17.61k stars 883 forks source link

Refactor CAggTimeBucketInfo to remove redundancy #6939

Closed fabriziomello closed 4 months ago

fabriziomello commented 4 months ago

Inside CAggTimebucketInfo data structure we're using several properties that actually is also part of ContinuousAggsBucketFunction, so refactor the code to use it and remove the properties redundancy.

Disable-check: force-changelog-file

codecov[bot] commented 4 months ago

Codecov Report

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

Project coverage is 81.70%. Comparing base (59f50f2) to head (d0173e8). Report is 167 commits behind head on main.

Files Patch % Lines
tsl/src/continuous_aggs/common.c 89.55% 0 Missing and 7 partials :warning:
tsl/src/continuous_aggs/create.c 80.00% 0 Missing and 2 partials :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #6939 +/- ## ========================================== + Coverage 80.06% 81.70% +1.63% ========================================== Files 190 198 +8 Lines 37181 36851 -330 Branches 9450 9637 +187 ========================================== + Hits 29770 30108 +338 + Misses 2997 2867 -130 + Partials 4414 3876 -538 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.