Fix #118 (partially): Don't print stacktrace on SQL syntax.

OndraZizka · Dec 1, 2024 · 1b5e659 · 1b5e659
1 parent cd1bc70
commit 1b5e659
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/src/main/kotlin/cz/dynawest/csvcruncher/CsvCruncherException.kt b/src/main/kotlin/cz/dynawest/csvcruncher/CsvCruncherException.kt
@@ -9,4 +9,9 @@ open class CsvCruncherException : RuntimeException {
 
 class CrucherConfigException : CsvCruncherException {
     constructor(message: String?) : super(message)
-}
+}
+
+class SqlSyntaxCruncherException : CsvCruncherException {
+    constructor(message: String?) : super(message)
+}
+
diff --git a/src/main/kotlin/cz/dynawest/csvcruncher/HsqlDbHelper.kt b/src/main/kotlin/cz/dynawest/csvcruncher/HsqlDbHelper.kt
@@ -97,6 +97,15 @@ class HsqlDbHelper(val jdbcConn: Connection) {
             jdbcConn.prepareStatement(sql)
         }
         catch (ex: SQLSyntaxErrorException) {
+            if (ex.message!!.contains("unexpected token:")) {
+                throw SqlSyntaxCruncherException("""
+                |    The SQL contains syntax error:
+                |    ${ex.message}
+                |    $sql
+                |    This may be your SQL error or caused by alteration by CsvCruncher. 
+                |    See https://github.com/OndraZizka/csv-cruncher/issues for known bugs.
+                |    """.trimMargin())
+            }
             if (ex.message!!.contains("object not found:")) {
                 throw throwHintForObjectNotFound(ex, this, sql)
             }