feat: Support date to timestamp cast#3383
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3383 +/- ##
============================================
+ Coverage 56.12% 60.02% +3.89%
- Complexity 976 1475 +499
============================================
Files 119 175 +56
Lines 11743 16184 +4441
Branches 2251 2688 +437
============================================
+ Hits 6591 9714 +3123
- Misses 4012 5114 +1102
- Partials 1140 1356 +216 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@parthchandra , @andygrove . Please take a look whenever you get a chance . |
|
I believe I can piggy back Date -> numeric types (which always return null) in thr PR or perhaps in a different PR if you think that is a good idea ? (Date -> Int is more a formality than adding additional logic) |
andygrove
left a comment
There was a problem hiding this comment.
LGTM, pending CI. Thanks @coderfender
|
Thank you for the quick approval @andygrove . Given that CI has passed , please merge the branch whenever you get a chance . Thank you |
Which issue does this PR close?
Adds cast support from date to timestamp
Closes #3400
Rationale for this change
What changes are included in this PR?
How are these changes tested?