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 <phillip@leblanc.tech>

* move all intervel test together

---------

Co-authored-by: Phillip LeBlanc <phillip@leblanc.tech>
y-f-u added a commit that referenced this pull request Jul 15, 2024
---------

Co-authored-by: Phillip LeBlanc <phillip@leblanc.tech>
y-f-u added a commit that referenced this pull request Jul 15, 2024
---------

Co-authored-by: Phillip LeBlanc <phillip@leblanc.tech>
y-f-u added a commit that referenced this pull request Jul 15, 2024
---------

Co-authored-by: Phillip LeBlanc <phillip@leblanc.tech>
y-f-u added a commit that referenced this pull request Jul 15, 2024
---------

Co-authored-by: Phillip LeBlanc <phillip@leblanc.tech>
y-f-u added a commit that referenced this pull request Jul 15, 2024
---------

Co-authored-by: Phillip LeBlanc <phillip@leblanc.tech>
y-f-u added a commit that referenced this pull request Jul 15, 2024
---------

Co-authored-by: Phillip LeBlanc <phillip@leblanc.tech>
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 <phillip@leblanc.tech>

* fmt

* clippy

* doc

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

Co-authored-by: Andrew Lamb <andrew@nerdnetworks.org>

* update the doc for CustomDialectBuilder

* fix doc test

---------

Co-authored-by: Phillip LeBlanc <phillip@leblanc.tech>
Co-authored-by: Andrew Lamb <andrew@nerdnetworks.org>
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