xiedeyantu commented on code in PR #4609:
URL: https://github.com/apache/calcite/pull/4609#discussion_r2496676572
##########
core/src/main/java/org/apache/calcite/sql/SqlTableRef.java:
##########
@@ -36,8 +36,15 @@ public class SqlTableRef extends SqlCall {
//~ Instance fields --------------------------------------------------------
- private final SqlIdentifier tableName;
- private final SqlNodeList hints;
+ /**
+ * Table name identifier that contains the fully qualified name of the table.
Review Comment:
Because I noticed that this is just a simple class which might be used in
various places. Can we be sure it refers to the fully qualified name here? As
you mentioned earlier, an identifier needs to include the full path to be
considered "fully qualified." I'm uncertain if this description might be
misleading.
--
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]
