dailai commented on code in PR #7814:
URL: https://github.com/apache/seatunnel/pull/7814#discussion_r1797502862


##########
seatunnel-connectors-v2/connector-jdbc/src/main/java/org/apache/seatunnel/connectors/seatunnel/jdbc/catalog/dm/DamengCatalog.java:
##########
@@ -184,4 +169,60 @@ public CatalogTable getTable(String sqlQuery) throws 
SQLException {
         Connection defaultConnection = getConnection(defaultUrl);
         return CatalogUtils.getCatalogTable(defaultConnection, sqlQuery, new 
DmdbTypeMapper());
     }
+
+    @Override
+    public CatalogTable getTable(TablePath tablePath)

Review Comment:
   The Super method used the sql to get the column medata data.
   
![image](https://github.com/user-attachments/assets/9bbcd4c0-8836-47f6-94eb-83f66f85583e)
   
   In dm database, if you want to achieve this function, you usually have to 
write a very complex sql which is not convenient. Therefore, I directly used 
the corresponding method of the jdbc driver to get the column information, 
which is accurate and easy to implement



-- 
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]

Reply via email to