Skip to content

[CALCITE-3397] AssertionError for interpretering multiset value - #1494

Closed
yanlin-Lynn wants to merge 1 commit into
apache:masterfrom
yanlin-Lynn:multiset
Closed

[CALCITE-3397] AssertionError for interpretering multiset value#1494
yanlin-Lynn wants to merge 1 commit into
apache:masterfrom
yanlin-Lynn:multiset

Conversation

@yanlin-Lynn

Copy link
Copy Markdown
Contributor

When interpretering sql select multiset['a', 'b', 'c'], got AssertionError.
Please refer https://issues.apache.org/jira/browse/CALCITE-3397 for details.

This PR tries to add a CollectNode to run Collect Relnode.

@hsyuan hsyuan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, except for minor comments

Comment thread core/src/main/java/org/apache/calcite/interpreter/CollectNode.java Outdated
@danny0405 danny0405 closed this in f7ff1c9 Oct 11, 2019
@yanlin-Lynn
yanlin-Lynn deleted the multiset branch October 11, 2019 05:25
XuQianJin-Stars pushed a commit to XuQianJin-Stars/calcite that referenced this pull request Nov 17, 2019
XuQianJin-Stars pushed a commit to XuQianJin-Stars/calcite that referenced this pull request Nov 20, 2019
XuQianJin-Stars pushed a commit to XuQianJin-Stars/calcite that referenced this pull request Nov 20, 2019
wangxlong pushed a commit to wangxlong/calcite that referenced this pull request Feb 13, 2020
jamesstarr pushed a commit to jamesstarr/calcite that referenced this pull request Aug 28, 2025
… Yanlin)

close apache#1494

Change-Id: I642acb68f83e5ff1fdbc9c6ee92c9b6eb55a894b
jamesstarr pushed a commit to jamesstarr/calcite that referenced this pull request Mar 16, 2026
… Yanlin)

close apache#1494

Change-Id: I642acb68f83e5ff1fdbc9c6ee92c9b6eb55a894b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants