Skip to content

Conversation

@y-f-u
Copy link

@y-f-u y-f-u commented Jul 11, 2024

Add interval style for dialect.

Keep dialect to be simple as a few toggles, and unparsing logic sit within expr

@y-f-u y-f-u changed the title wip custom scalar_to_sql from dialect Jul 12, 2024
@y-f-u y-f-u changed the title custom scalar_to_sql from dialect dialect with interval style Jul 12, 2024
@y-f-u y-f-u marked this pull request as ready for review July 12, 2024 06:55
@y-f-u y-f-u self-assigned this Jul 12, 2024
@y-f-u y-f-u merged commit dfb42ad into spiceai-39 Jul 15, 2024
@y-f-u y-f-u deleted the unparser-rule-for-interval branch July 15, 2024 02:15
phillipleblanc added a commit that referenced this pull request Jul 15, 2024
* wip

* add interval style

* format

* Apply suggestions from code review

Co-authored-by: Phillip LeBlanc <[email protected]>

* move all intervel test together

---------

Co-authored-by: Phillip LeBlanc <[email protected]>
y-f-u added a commit that referenced this pull request Jul 15, 2024
---------

Co-authored-by: Phillip LeBlanc <[email protected]>
y-f-u added a commit that referenced this pull request Jul 15, 2024
---------

Co-authored-by: Phillip LeBlanc <[email protected]>
y-f-u added a commit that referenced this pull request Jul 15, 2024
---------

Co-authored-by: Phillip LeBlanc <[email protected]>
y-f-u added a commit that referenced this pull request Jul 15, 2024
---------

Co-authored-by: Phillip LeBlanc <[email protected]>
y-f-u added a commit that referenced this pull request Jul 15, 2024
---------

Co-authored-by: Phillip LeBlanc <[email protected]>
y-f-u added a commit that referenced this pull request Jul 15, 2024
---------

Co-authored-by: Phillip LeBlanc <[email protected]>
sgrebnov pushed a commit that referenced this pull request Jul 16, 2024
… `interval` (apache#11466)

* Unparser rule for datatime cast (#10)

* use timestamp as the identifier for date64

* rename

* implement CustomDialectBuilder

* fix

* dialect with interval style (#11)

---------

Co-authored-by: Phillip LeBlanc <[email protected]>

* fmt

* clippy

* doc

* Update datafusion/sql/src/unparser/expr.rs

Co-authored-by: Andrew Lamb <[email protected]>

* update the doc for CustomDialectBuilder

* fix doc test

---------

Co-authored-by: Phillip LeBlanc <[email protected]>
Co-authored-by: Andrew Lamb <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants