-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
HugeInt support in TypeSignature conversion (#6537) #6799
HugeInt support in TypeSignature conversion (#6537) #6799
Conversation
✅ Deploy Preview for meta-velox canceled.
|
This pull request was exported from Phabricator. Differential Revision: D49212995 |
.returnType("hugeint") | ||
.argumentType("hugeint") | ||
.build(); | ||
testSignatureBinder(signature, {HUGEINT()}, HUGEINT()); |
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.
Thank you for adding this test! Could you also test that if a function signature is T -> T, whether {HUGEINT()} argument type can be correctly resolved?
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.
added
This pull request was exported from Phabricator. Differential Revision: D49212995 |
Summary: This came up while interpreting Velox function signatures. Eg UDFs using HugeInt. Reviewed By: darrenfu, pedroerp Differential Revision: D49212995
This pull request was exported from Phabricator. Differential Revision: D49212995 |
Recreate new PR for linux build issues. |
This pull request was exported from Phabricator. Differential Revision: D49212995 |
2 similar comments
This pull request was exported from Phabricator. Differential Revision: D49212995 |
This pull request was exported from Phabricator. Differential Revision: D49212995 |
Summary: Pull Request resolved: facebookincubator#6799 This came up while interpreting Velox function signatures. Eg UDFs using HugeInt. Reviewed By: darrenfu, pedroerp Differential Revision: D49212995 fbshipit-source-id: 7362e1c22af33ddb355757545cee2dbf5da4f25a
Summary: This came up while interpreting Velox function signatures. Eg UDFs using HugeInt. Reviewed By: darrenfu, pedroerp Differential Revision: D49212995
Summary: Pull Request resolved: #6799 This came up while interpreting Velox function signatures. Eg UDFs using HugeInt. Reviewed By: darrenfu, pedroerp Differential Revision: D49212995 fbshipit-source-id: 5f39548134ab13d0ca6df11ff0895245c7f4165c
Summary: Pull Request resolved: facebookincubator#6799 This came up while interpreting Velox function signatures. Eg UDFs using HugeInt. Reviewed By: darrenfu, pedroerp Differential Revision: D49212995 fbshipit-source-id: 5f39548134ab13d0ca6df11ff0895245c7f4165c
Summary:
This came up while interpreting Velox function signatures. Eg UDFs using HugeInt.
Reviewed By: darrenfu, pedroerp
Differential Revision: D49212995