cube icon indicating copy to clipboard operation
cube copied to clipboard

DuckDB dates interpreted as strings instead of timestamp

Open ygg opened this issue 2 years ago β€’ 2 comments

Check List

  • [ ] Tests has been run in packages where changes made if available
  • [ ] Linter has been run for changed code
  • [ ] Tests for the changes have been added if not covered yet
  • [ ] Docs have been added / updated if required

Issue Reference this PR resolves

Description of Changes Made (if issue reference is not provided)

CubeJS was treating DuckDB TIMESTAMPTZ data type as a string rather than a timestamp. This created problems with pre-aggregation processes, especially when trying to group data by units of time like months, weeks, etc.

ygg avatar Jul 04 '23 13:07 ygg

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 47.30%. Comparing base (b57ddc3) to head (04e3fcf). Report is 2180 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6782      +/-   ##
==========================================
- Coverage   47.32%   47.30%   -0.03%     
==========================================
  Files         155      155              
  Lines       20438    20438              
  Branches     5167     5167              
==========================================
- Hits         9673     9668       -5     
- Misses      10449    10454       +5     
  Partials      316      316              
Flag Coverage Ξ”
cube-backend 47.30% <ΓΈ> (-0.03%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Jul 04 '23 16:07 codecov[bot]

@ygg Thanks for contributing! Could you please add an integration or smoke test for that?

paveltiunov avatar Aug 30 '23 04:08 paveltiunov

Hi @ygg Can you elaborate when/where cube treats date_trunc's as strings rather then timestamps?

KSDaemon avatar Jun 19 '25 12:06 KSDaemon