-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Running branch for Clickhouse pushdowns #23460
base: master
Are you sure you want to change the base?
Running branch for Clickhouse pushdowns #23460
Conversation
bc141cf
to
4195edb
Compare
4195edb
to
2fcedb9
Compare
344a9b5
to
c7c253b
Compare
c3cd59d
to
bacc519
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ssheikin is this PR ready for review? I see some commented out codes as well. Could you please update the description of the PR and commit message?
This is not ready to be merged, but as almost any PR it's ready for review 😉 |
d726ebd
to
16d241f
Compare
5ae0bf5
to
f2ba3ee
Compare
79c478d
to
970eda6
Compare
680ac7f
to
aa0293c
Compare
aa0293c
to
b1bc584
Compare
This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua |
TIMESTAMP(p) TIMESTAMP(p) WITH TIME ZONE
b1bc584
to
50d6538
Compare
Description
Smaller PRs will be chopped from this uber-PR.
TODO for varchars
Additional context and related issues
VARCHAR
PRs:
DATETIME
PRs:
ClickHouse docs:
Release notes
( ) This is not user-visible or is docs only, and no release notes are required.
(x) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text: