Github user HanumathRao commented on the issue:

    https://github.com/apache/drill/pull/996
  
    @arina-ielchiieva I think this check shouldn't cause much of a performance 
impact as it is in parser code and also it is checked right now by the DRILL's 
custom overload of getTable. If this function throws an exception then it is 
caught and reported to the user. I think there can be improvement to not check 
this by every code path(i.e valid code path as well) and only check when 
super.getTable returns null, in this way we can report error only when we 
couldn't find a table. Please let me know if this approach seems reasonable, so 
that I can go ahead and change the code accordingly.


---

Reply via email to