Skip to content

[CALCITE-4999] ARRAY, MULTISET functions should return a collection of scalars if a sub-query returns 1 column#2712

Merged
rubenada merged 1 commit intoapache:mainfrom
dssysolyatin:CALCITE-4999
Aug 30, 2022
Merged

[CALCITE-4999] ARRAY, MULTISET functions should return a collection of scalars if a sub-query returns 1 column#2712
rubenada merged 1 commit intoapache:mainfrom
dssysolyatin:CALCITE-4999

Conversation

@dssysolyatin
Copy link
Contributor

No description provided.

@rubenada
Copy link
Contributor

rubenada commented Feb 4, 2022

Could you please provide a unit test with a SQL query? Probably the best option would be adding the problematic query into big-query.iq

@rubenada
Copy link
Contributor

rubenada commented Feb 4, 2022

Kindly reminder: normally the commit message should be the same as the Jira title (in case of a bug, it should normally describe the bug) https://calcite.apache.org/develop/#contributing

@julianhyde
Copy link
Contributor

This proposed change is not an "improvement".

@rubenada rubenada added the discussion-in-jira There's open discussion in JIRA to be resolved before proceeding with the PR label Feb 8, 2022
@dssysolyatin dssysolyatin changed the title [CALCITE-4999] Improve least restrictive type inference [CALCITE-4999] ARRAY function should return an array of scalars if subquery returns 1 column Apr 21, 2022
@dssysolyatin dssysolyatin force-pushed the CALCITE-4999 branch 7 times, most recently from de9ca27 to 888a134 Compare April 26, 2022 08:18
@dssysolyatin dssysolyatin changed the title [CALCITE-4999] ARRAY function should return an array of scalars if subquery returns 1 column [CALCITE-4999] ARRAY, MULTISET functions should return a collection of scalars if a sub-query returns 1 column Jul 14, 2022
@rubenada rubenada added LGTM-will-merge-soon Overall PR looks OK. Only minor things left. and removed discussion-in-jira There's open discussion in JIRA to be resolved before proceeding with the PR labels Aug 29, 2022
@rubenada rubenada merged commit 71e30e2 into apache:main Aug 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

LGTM-will-merge-soon Overall PR looks OK. Only minor things left.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants