fix(cubesql): Push down UTCTIMESTAMP (CURRENT_TIMESTAMP)#11195
Conversation
|
Claude finished @MazterQyou's task in 1m 20s —— View job Code ReviewLGTM — 0 blocking issues, 2 minor notes. Expand for details.
Summary Straightforward, well-scoped change: each dialect-specific Per-dialect correctness (spot check)
Minor notes (non-blocking)
Nothing else stands out — nice, focused patch. |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #11195 +/- ##
=======================================
Coverage 79.31% 79.32%
=======================================
Files 472 472
Lines 96921 96953 +32
Branches 3539 3539
=======================================
+ Hits 76877 76908 +31
- Misses 19538 19539 +1
Partials 506 506
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
3f7da9f to
66baf07
Compare
|
Claude finished @MazterQyou's task in 38s —— View job Code ReviewLGTM — 0 blocking issues, 3 minor notes. Expand for details.
Summary Focused, well-scoped change. Each dialect-specific Per-dialect correctness
Minor notes (non-blocking)
Nothing else stands out — nice, focused patch. • Branch: |
Check List
Description of Changes Made
This PR adds
UTCTIMESTAMPSQL push down templates soCURRENT_TIMESTAMPcan use SQL push down when needed. Related test is included.