luozenglin commented on a change in pull request #6374:
URL: https://github.com/apache/incubator-doris/pull/6374#discussion_r683083208
##########
File path: fe/fe-core/src/main/java/org/apache/doris/analysis/Analyzer.java
##########
@@ -618,10 +617,6 @@ public SlotDescriptor registerColumnRef(TableName tblName,
String colName) throw
if (newTblName == null) {
Review comment:
When `newTblName ` is not empty, pass`newTblName ` to
`resolveColumnRef()` to construct `TupleDescriptor
##########
File path: fe/fe-core/src/main/java/org/apache/doris/analysis/Analyzer.java
##########
@@ -553,7 +552,7 @@ public TableRef resolveTableRef(TableRef tableRef) throws
AnalysisException {
ErrorReport.reportAnalysisException(ErrorCode.ERR_BAD_TABLE_STATE,
"RESTORING");
}
- TableName tblName = new TableName(database.getFullName(),
table.getName());
+ TableName tblName = new TableName(dbName, tableName.getTbl());
Review comment:
`tableName.getTbl()` stores the table name specified by the user in the
`from` statement. In the case of case-sensitive table names, the value of
`tableName.getTbl()` is the same as `table.getName()`. However, since the
system view is not case-sensitive, `table.getName()` gets the lowercase view
name, which may not be the same as the user's reference to the table name,
causing the table name not to be found in registerColumnRef().
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
