Skip to content

Conversation

aceforeverd
Copy link
Collaborator

@aceforeverd aceforeverd commented Sep 9, 2023

For datadiff function with at least one string parameter, the string date is allowed to be before year 1900.
NOTE: openmldb date type only supports year after 1900.

@github-actions github-actions bot added the execute-engine hybridse sql engine label Sep 9, 2023
@codecov
Copy link

codecov bot commented Sep 9, 2023

Codecov Report

Patch coverage: 94.59% and project coverage change: +38.20% 🎉

Comparison is base (b7d6edc) 37.14% compared to head (4e14ea0) 75.35%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@              Coverage Diff              @@
##               main    #3499       +/-   ##
=============================================
+ Coverage     37.14%   75.35%   +38.20%     
  Complexity      414      414               
=============================================
  Files           179      707      +528     
  Lines         10364   128677   +118313     
  Branches       1191     1191               
=============================================
+ Hits           3850    96962    +93112     
- Misses         6273    31474    +25201     
  Partials        241      241               
Files Changed Coverage Δ
hybridse/src/udf/udf.h 96.00% <ø> (ø)
hybridse/src/udf/udf.cc 86.44% <91.66%> (ø)
hybridse/src/codegen/udf_ir_builder_test.cc 96.76% <100.00%> (ø)
hybridse/src/udf/default_udf_library.cc 97.27% <100.00%> (ø)

... and 524 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@aceforeverd aceforeverd force-pushed the feat-datediff-date2 branch 2 times, most recently from 80a6da2 to 6e8cc2e Compare September 11, 2023 03:59
Copy link
Collaborator

@tobegit3hub tobegit3hub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aceforeverd aceforeverd merged commit 116fbf5 into 4paradigm:main Oct 7, 2023
@aceforeverd aceforeverd deleted the feat-datediff-date2 branch October 7, 2023 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
execute-engine hybridse sql engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants