Github user HanumathRao commented on a diff in the pull request:
https://github.com/apache/drill/pull/996#discussion_r145268123
--- 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 --
Thank you for the review. I agree that this should ideally be handled at
Calcite layer. I also think that even after Calcite providing this
functionality there should be some customization that needs to be done as we
understand the context better than Calcite. Once the calcite fixes this issue
then we can always change the code accordingly.
---
