[GitHub] [doris] morningman commented on a diff in pull request #24520: [improvement](jdbc catalog) when `lower_case_table_names` of jdbc catalog properties is set to `true`, use the real table name to query the jdbc data source

[via GitHub]

morningman commented on code in PR #24520:
URL: https://github.com/apache/doris/pull/24520#discussion_r1331084308


##########
fe/fe-core/src/main/java/org/apache/doris/datasource/jdbc/client/JdbcClient.java:
##########
@@ -367,6 +367,20 @@ public List<Column> getColumnsFromJdbc(String dbName, 
String tableName) {
         return dorisTableSchema;
     }
 
+    public String getRealDatabaseName(String dbname) {
+        if (lowerDBToRealDB.isEmpty()) {
+            getDatabaseNameList();

Review Comment:
   There may be concurrency issue



-- 
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: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org