Merge pull request #263 from datafuselabs/fix/detail-exception

fix: add more exception info
databendlabs · Sep 9, 2024 · df27b70 · df27b70
2 parents a1c0090 + 715a9b5
commit df27b70
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/databend-jdbc/src/main/java/com/databend/jdbc/DatabendConnection.java b/databend-jdbc/src/main/java/com/databend/jdbc/DatabendConnection.java
@@ -871,7 +871,6 @@ public void uploadStream(String stageName, String destPrefix, InputStream inputS
             }
         } catch (RuntimeException e) {
             System.out.println(e.getMessage());
-            // For datax batch insert test, do not throw exception
             throw new SQLException(e);
         } catch (IOException e) {
             logger.warning("failed to upload input stream, file size is:" + fileSize / 1024.0 + e.getMessage());

diff --git a/databend-jdbc/src/main/java/com/databend/jdbc/PresignContext.java b/databend-jdbc/src/main/java/com/databend/jdbc/PresignContext.java
@@ -60,7 +60,7 @@ public static PresignContext getPresignContext(DatabendConnection connection, Pr
                 throw new SQLException("Failed to get presign url");
             }
         } catch (Throwable e) {
-            throw new SQLException("Failed to do presign");
+            throw new SQLException("Failed to do presign. Exception: " + e.toString(), e);
         } finally {
             statement.close();
         }