[GitHub] [doris] morningman commented on a diff in pull request #22930: [improve](jdbc catalog) Add more detailed exception logging to the jdbc catalog

["morningman (via GitHub)" <gi...@apache.org>]

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


##########
fe/fe-core/src/main/java/org/apache/doris/datasource/jdbc/client/JdbcClient.java:
##########
@@ -160,7 +162,8 @@ public Connection getConnection() throws JdbcClientException {
         try {
             conn = dataSource.getConnection();
         } catch (Exception e) {
-            throw new JdbcClientException("Can not connect to jdbc", e);
+            throw new JdbcClientException("Can not connect to jdbc due to error: %s, catalog name: %s", e,

Review Comment:
   `e` should be at the end.



-- 
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-unsubscribe@doris.apache.org

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


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