-
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
Support UNKNOWN type in Map function #9410
Conversation
This pull request was exported from Phabricator. Differential Revision: D55902535 |
✅ Deploy Preview for meta-velox canceled.
|
Summary: Currently the key type of `MAP` is specified as `knownTypeVariable` which refuses to bind `UNKNOWN` type, while in Presto `MAP(UNKNOWN, UNKNOWN)` is valid. Differential Revision: D55902535
This pull request was exported from Phabricator. Differential Revision: D55902535 |
Summary: Currently the key type of `MAP` is specified as `knownTypeVariable` which refuses to bind `UNKNOWN` type, while in Presto `MAP(UNKNOWN, UNKNOWN)` is valid. Differential Revision: D55902535
This pull request was exported from Phabricator. Differential Revision: D55902535 |
Summary: Currently the key type of `MAP` is specified as `knownTypeVariable` which refuses to bind `UNKNOWN` type, while in Presto `MAP(UNKNOWN, UNKNOWN)` is valid. Differential Revision: D55902535
This pull request was exported from Phabricator. Differential Revision: D55902535 |
This pull request has been merged in 89de5d3. |
Conbench analyzed the 1 benchmark run on commit There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
Summary: Pull Request resolved: facebookincubator#9410 Currently the key type of `MAP` is specified as `knownTypeVariable` which refuses to bind `UNKNOWN` type, while in Presto `MAP(UNKNOWN, UNKNOWN)` is valid. Reviewed By: kagamiori Differential Revision: D55902535 fbshipit-source-id: 8c57736859d9f726b1cb7aa43cec74d60990c630
Summary: Pull Request resolved: facebookincubator#9410 Currently the key type of `MAP` is specified as `knownTypeVariable` which refuses to bind `UNKNOWN` type, while in Presto `MAP(UNKNOWN, UNKNOWN)` is valid. Reviewed By: kagamiori Differential Revision: D55902535 fbshipit-source-id: 8c57736859d9f726b1cb7aa43cec74d60990c630
Summary: Currently the key type of
MAP
is specified asknownTypeVariable
which refuses to bindUNKNOWN
type, while in PrestoMAP(UNKNOWN, UNKNOWN)
is valid.Differential Revision: D55902535