substrait-io / substrait

A cross platform way to express data transformation, relational algebra, standardized record expression and plans.
https://substrait.io
Apache License 2.0
1.17k stars 150 forks source link

feat: add current_date and current_timestamp functions #524

Closed MasseGuillaume closed 1 month ago

MasseGuillaume commented 1 year ago

Ah got it, it makes sense. In my use case, I use substrait to serialize the SQL so a rich UI can display a query plan to the user. In this case, the query is not yet executed, so it would not make sense to constant fold the current_time/current_date. What's missing is substrait is an optional function result for non-deterministic functions.

EpsilonPrime commented 1 year ago

Similar functionality will be added in #528 so I believe we can close this PR.

westonpace commented 1 year ago

@EpsilonPrime did you perhaps mean #548 ?

EpsilonPrime commented 1 year ago

Ah, yes, thanks for the correction!

jacques-n commented 1 month ago

Closing in preference to #548