-
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 arbitrary aggregation #6557
Conversation
✅ Deploy Preview for meta-velox canceled.
|
This pull request was exported from Phabricator. Differential Revision: D49241110 |
Summary: This change adds support for the UNKNOWN type in arbitrary aggregation. It mostly comes up when a query includes arbitrary(NULL) as a means of getting a NULL from an aggregate without needing to group by it. It's fairly trivial since NonNumericArbitrary supports it directly. Differential Revision: D49241110
2ca00e0
to
a492a3f
Compare
This pull request was exported from Phabricator. Differential Revision: D49241110 |
Summary: This change adds support for the UNKNOWN type in arbitrary aggregation. It mostly comes up when a query includes arbitrary(NULL) as a means of getting a NULL from an aggregate without needing to group by it. It's fairly trivial since NonNumericArbitrary supports it directly. Reviewed By: mbasmanova Differential Revision: D49241110
Summary: This change adds support for the UNKNOWN type in arbitrary aggregation. It mostly comes up when a query includes arbitrary(NULL) as a means of getting a NULL from an aggregate without needing to group by it. It's fairly trivial since NonNumericArbitrary supports it directly. Reviewed By: mbasmanova Differential Revision: D49241110
a492a3f
to
ae3c213
Compare
This pull request was exported from Phabricator. Differential Revision: D49241110 |
ae3c213
to
fd1dd09
Compare
This pull request was exported from Phabricator. Differential Revision: D49241110 |
This pull request has been merged in 926a49f. |
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#6557 This change adds support for the UNKNOWN type in arbitrary aggregation. It mostly comes up when a query includes arbitrary(NULL) as a means of getting a NULL from an aggregate without needing to group by it. It's fairly trivial since NonNumericArbitrary supports it directly. Reviewed By: mbasmanova Differential Revision: D49241110 fbshipit-source-id: f90b832f46ac121b67bf7d8bdf40a1bbaa9f70da
Summary: Pull Request resolved: facebookincubator#6557 This change adds support for the UNKNOWN type in arbitrary aggregation. It mostly comes up when a query includes arbitrary(NULL) as a means of getting a NULL from an aggregate without needing to group by it. It's fairly trivial since NonNumericArbitrary supports it directly. Reviewed By: mbasmanova Differential Revision: D49241110 fbshipit-source-id: f90b832f46ac121b67bf7d8bdf40a1bbaa9f70da
Summary: Pull Request resolved: facebookincubator#6557 This change adds support for the UNKNOWN type in arbitrary aggregation. It mostly comes up when a query includes arbitrary(NULL) as a means of getting a NULL from an aggregate without needing to group by it. It's fairly trivial since NonNumericArbitrary supports it directly. Reviewed By: mbasmanova Differential Revision: D49241110 fbshipit-source-id: f90b832f46ac121b67bf7d8bdf40a1bbaa9f70da
Summary: Pull Request resolved: facebookincubator#6557 This change adds support for the UNKNOWN type in arbitrary aggregation. It mostly comes up when a query includes arbitrary(NULL) as a means of getting a NULL from an aggregate without needing to group by it. It's fairly trivial since NonNumericArbitrary supports it directly. Reviewed By: mbasmanova Differential Revision: D49241110 fbshipit-source-id: f90b832f46ac121b67bf7d8bdf40a1bbaa9f70da
Summary:
This change adds support for the UNKNOWN type in arbitrary aggregation, it's fairly trivial since
NonNumericArbitrary supports it directly.
Differential Revision: D49241110