normanj-bitquill commented on code in PR #3802:
URL: https://github.com/apache/calcite/pull/3802#discussion_r1626659020
##########
core/src/main/java/org/apache/calcite/sql/validate/DelegatingScope.java:
##########
@@ -297,8 +298,17 @@ protected void addColumnNames(
namespace = map.get(tableName).namespace;
break;
default:
- throw validator.newValidationError(identifier,
- RESOURCE.columnAmbiguous(columnName));
+ // Allow if all namespaces are the same and the tables are not null
Review Comment:
@YiwenWu I'm not sure. If we have a query such as:
```
select x, * from foo f join bar b using (x)
```
Does it matter if we choose `foo.x` or `bar.x` in this context? Due to the
join, they should have the same value.
--
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]
