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

Extend wrapDef to def and overdef #2060

Merged
merged 9 commits into from
Dec 20, 2024
Merged

Conversation

mtoy-googly-moogly
Copy link
Collaborator

No description provided.

@mtoy-googly-moogly mtoy-googly-moogly merged commit 01b7333 into array-func-cleanup Dec 20, 2024
9 checks passed
@mtoy-googly-moogly mtoy-googly-moogly deleted the def-overdef branch December 20, 2024 19:04
mtoy-googly-moogly added a commit that referenced this pull request Dec 21, 2024
* make wrapDef a util

* move repeat out of standard library

* cleanup wrapDef a little more

* even more wrapDef cleanup

* fix oopsie

* maybe cleaner still?

* fix find/replace mistake

* rename arg

* make all param names into strings

* fix trino reverse

* moved todos to docs issue

* remove extra test for % operator

* put the cast back into trino reverse

* go ahead and test reverse for arrays on presto

* try again to get reverse with generic correct

* shared string_reverse

* maybe finally fix reverse?

* use defwrap to cleanup some errors

* another minor code tweak to wrapDef

* -cleverness, +readability

* add comment

* comment edit

* comment explaining dialect split

* fix T comment

* Extend wrapDef to def and overdef (#2060)

* experiment with overdef

* fix overdef()

* mark bitwise_aggs as symmetric

* move all the hll functiosn to ... def

* make preto-only all def()

* delete overdef

* stop using def for hll functions

* fix presto funcs

* cleanup sequence()
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.

1 participant