morrySnow commented on code in PR #61918:
URL: https://github.com/apache/doris/pull/61918#discussion_r3019639354
##########
fe/fe-core/src/main/java/org/apache/doris/info/TableNameInfo.java:
##########
@@ -52,6 +48,16 @@ public TableNameInfo() {
}
+ /**
+ * Create a TableNameInfo from a Database and table name.
+ * Uses the database's catalog name, falling back to INTERNAL_CATALOG_NAME.
+ */
+ public static TableNameInfo of(Database db, String tableName) {
Review Comment:
do not add this in TableNameInfo, because the target is to decouple this
class from Database. create a new util calss to do it. and then, in this pr,
there are so many similar code block. could you simplify them?
--
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]
