RD-11280: a schema does not exist error isn't reported to the user (#…

…468)
raw-labs · Aug 1, 2024 · 5d81a44 · 5d81a44
1 parent 87bde80
commit 5d81a44
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/sql-client/src/main/scala/raw/client/sql/NamedParametersPreparedStatement.scala b/sql-client/src/main/scala/raw/client/sql/NamedParametersPreparedStatement.scala
@@ -566,7 +566,7 @@ class NamedParametersPreparedStatement(
   }
 
   private def asErrorString(ex: PSQLException): Option[String] = {
-    if (Set("42", "22", "0A").exists(ex.getSQLState.startsWith)) {
+    if (Set("42", "22", "0A", "3F").exists(ex.getSQLState.startsWith)) {
       // syntax error / semantic error
       val psqlError = Option(ex.getServerErrorMessage) // getServerErrorMessage can be null!
       val error = psqlError.map(_.getMessage).getOrElse(ex.getMessage)

diff --git a/sql-client/src/test/scala/raw/client/sql/TestSqlCompilerServiceAirports.scala b/sql-client/src/test/scala/raw/client/sql/TestSqlCompilerServiceAirports.scala
@@ -956,4 +956,17 @@ class TestSqlCompilerServiceAirports
     assert(v.messages.size == 1)
     assert(v.messages(0).message == "non-executable code")
   }
+
+  test("""select
+    |  airport_id,
+    |  city,
+    |  country.name 'cname'
+    |from
+    |  example.airports
+    |limit 10;
+    |""".stripMargin) { t =>
+    val v = compilerService.validate(t.q, asJson())
+    assert(v.messages.size == 1)
+    assert(v.messages(0).message == "schema \"country\" does not exist")
+  }
 }