-
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
Fix: support unknown in map_from_entries function #6353
Conversation
This pull request was exported from Phabricator. Differential Revision: D48837795 |
✅ Deploy Preview for meta-velox canceled.
|
This pull request was exported from Phabricator. Differential Revision: D48837795 |
…6353) Summary: Pull Request resolved: facebookincubator#6353 presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Differential Revision: D48837795 fbshipit-source-id: 2d9cc9644b3b917133efed9130a7ff9e085848bd
818ffe0
to
7cb85f1
Compare
This pull request was exported from Phabricator. Differential Revision: D48837795 |
7cb85f1
to
6a8e597
Compare
…6353) Summary: Pull Request resolved: facebookincubator#6353 presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Differential Revision: D48837795 fbshipit-source-id: b581359fbf96684fe85f5fe00ac12e3ce15d5ab3
This pull request was exported from Phabricator. Differential Revision: D48837795 |
…6353) Summary: Pull Request resolved: facebookincubator#6353 presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Differential Revision: D48837795 fbshipit-source-id: cf5a3c8fcf9ed3f55764789ed712febb15a2349a
6a8e597
to
34bd6f7
Compare
…6353) Summary: presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Differential Revision: D48837795
34bd6f7
to
266cf26
Compare
This pull request was exported from Phabricator. Differential Revision: D48837795 |
…6353) Summary: presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Differential Revision: D48837795
266cf26
to
32e504e
Compare
This pull request was exported from Phabricator. Differential Revision: D48837795 |
…6353) Summary: presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Differential Revision: D48837795
32e504e
to
291288f
Compare
This pull request was exported from Phabricator. Differential Revision: D48837795 |
…6353) Summary: presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Differential Revision: D48837795
291288f
to
8a1614b
Compare
This pull request was exported from Phabricator. Differential Revision: D48837795 |
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.
Thanks.
rows.end(), | ||
sizes, | ||
offsets, | ||
BaseVector::create(UNKNOWN(), 0, context.pool()), |
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.
You can pass nullptr here
This pull request was exported from Phabricator. Differential Revision: D48837795 |
…6353) Summary: Pull Request resolved: facebookincubator#6353 presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Reviewed By: mbasmanova Differential Revision: D48837795 fbshipit-source-id: 1b78a2ed4fce651f644f35f399f0e24574149f1d
8a1614b
to
7cc08a9
Compare
…6353) Summary: presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Reviewed By: mbasmanova Differential Revision: D48837795
7cc08a9
to
a12b3a9
Compare
This pull request was exported from Phabricator. Differential Revision: D48837795 |
…6353) Summary: presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Reviewed By: mbasmanova Differential Revision: D48837795
a12b3a9
to
f3b1172
Compare
This pull request was exported from Phabricator. Differential Revision: D48837795 |
…6353) Summary: presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Reviewed By: mbasmanova Differential Revision: D48837795
f3b1172
to
b7102cc
Compare
This pull request was exported from Phabricator. Differential Revision: D48837795 |
…6353) Summary: presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Reviewed By: mbasmanova Differential Revision: D48837795
b7102cc
to
a5daed9
Compare
This pull request was exported from Phabricator. Differential Revision: D48837795 |
…6353) Summary: presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Reviewed By: mbasmanova Differential Revision: D48837795
a5daed9
to
5602309
Compare
This pull request was exported from Phabricator. Differential Revision: D48837795 |
This pull request has been merged in a3c4aac. |
Conbench analyzed the 1 benchmark run on commit There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
…6353) Summary: Pull Request resolved: facebookincubator#6353 presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Reviewed By: mbasmanova Differential Revision: D48837795 fbshipit-source-id: 372097d333f9f90a5de4507c584d29388dd18d95
…6353) Summary: Pull Request resolved: facebookincubator#6353 presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Reviewed By: mbasmanova Differential Revision: D48837795 fbshipit-source-id: 372097d333f9f90a5de4507c584d29388dd18d95
…6353) Summary: Pull Request resolved: facebookincubator#6353 presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Reviewed By: mbasmanova Differential Revision: D48837795 fbshipit-source-id: 372097d333f9f90a5de4507c584d29388dd18d95
…6353) Summary: Pull Request resolved: facebookincubator#6353 presto support `` try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)])) `` and `` try (select MAP_FROM_ENTRIES(ARRAY[null])) `` Reviewed By: mbasmanova Differential Revision: D48837795 fbshipit-source-id: 372097d333f9f90a5de4507c584d29388dd18d95
Summary:
presto support
try (select MAP_FROM_ENTRIES(ARRAY[Row(null, null)]))
and
try (select MAP_FROM_ENTRIES(ARRAY[null]))
Differential Revision: D48837795