Jinfeng Ni created DRILL-3666:
---------------------------------
Summary: Replace assert uses in Drill's interpreter to catch error
when it happens.
Key: DRILL-3666
URL: https://issues.apache.org/jira/browse/DRILL-3666
Project: Apache Drill
Issue Type: Bug
Components: Query Planning & Optimization
Affects Versions: 1.1.0
Reporter: Jinfeng Ni
Assignee: Jinfeng Ni
Some code in interpreter uses assert to check certain conditions. However, if
assertion enabled (-ea) is not turned on, such checking would skip. That might
cause issues, since the caller may not know an error has happened in the
interpreter, and will continue the execution. This has caused issue in
partition pruning recently, when a bug in partition pruning passes an invalid
expression to interpreter, yet the assert checking was skipped and end up with
incorrect query result.
In stead, we should remove assert in interpreter, and make sure throw exception
when certain conditions are not met. It's up the caller to decide what to do in
the case of exception.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
