Github user arina-ielchiieva commented on a diff in the pull request:
https://github.com/apache/drill/pull/996#discussion_r145231215
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/SqlConverter.java
---
@@ -481,6 +485,19 @@ public RelOptTableImpl getTable(final List<String>
names) {
.message("Temporary tables usage is disallowed. Used temporary
table name: %s.", names)
.build(logger);
}
+
+ // Check the schema and throw a valid SchemaNotFound exception
instead of TableNotFound exception.
--- End diff --
Does it mean that Calcite instead of returning schema not found exception
returns table not found exception?
Per my understanding this PR customizes Drill but what if we go different
path and enhance Calcite (or may be this is already done in newer Calcite
versions)?
---
