Skip to content
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

fix(trino|presto): adding cast support for "hyperloglog" column type #3405

Merged
merged 5 commits into from
May 3, 2024

Conversation

uncledata
Copy link
Contributor

No description provided.

@uncledata uncledata changed the title fix(trino): adding cast support for "hyperloglog" column type fix(trino|presto): adding cast support for "hyperloglog" column type May 3, 2024
@tobymao
Copy link
Owner

tobymao commented May 3, 2024

this looks incomplete, can you add a test case following the existing patterns that we have?

@uncledata
Copy link
Contributor Author

hm, somehow not all moved. One sec

sqlglot/tokens.py Outdated Show resolved Hide resolved
hyperloglog->token HLLSKETCH
@uncledata
Copy link
Contributor Author

Thanks for the support! hopefully next incoming PR's will be smoother.

@georgesittas georgesittas merged commit 24e1115 into tobymao:main May 3, 2024
5 checks passed
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.

None yet

3 participants