sbroeder commented on code in PR #5271:
URL: https://github.com/apache/calcite/pull/5271#discussion_r4138906788
##########
core/src/main/java/org/apache/calcite/sql/validate/AliasNamespace.java:
##########
@@ -111,6 +111,7 @@ protected AliasNamespace(
rowType.getFieldList().get(0).getType())
.build();
aliasedType = node.getKind() == SqlKind.COLLECTION_TABLE
+ || (node.getKind() == SqlKind.UNNEST &&
isUnnestOfStructArray((SqlCall) node))
? new SingleColumnAliasRelDataType(rowType, singleColumnAlias) :
singleColumnAlias;
Review Comment:
You're right. To be precise: the field name is what star-expansion and
NATURAL JOIN use, but the alias also still names the column directly and is
accepted in USING.
That's consistent with how the existing
COLLECTION_TABLE/single-column-UNNEST case. Added the USING (d) case to the
test, alongside the existing USING (a) one.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
