----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19706/ -----------------------------------------------------------
Review request for drill. Repository: drill-git Description ------- This patch fixes the issue in the ANTLR expression parser used by Drill. When the parser sees parsing error in the expression in serialized logical plan/physical plan, it only will display the error message in the console, and continue the execution. The result logical expression will be either invalid or incomplete, which eventually hit error in Drill's run-time phase. This makes debuggging the error more difficulty. Modify the ANTLR parser, such that it will catch the RecognitionException thrown by parser, and stop the execution. Diffs ----- common/src/main/antlr3/org/apache/drill/common/expression/parser/ExprParser.g edb8507 exec/java-exec/src/test/java/org/apache/drill/exec/expr/ExpressionTest.java 3491e7a pom.xml c4bc9c9 Diff: https://reviews.apache.org/r/19706/diff/ Testing ------- Add two unit tests in ExpressionTest.java. One will raise the exception. The other one is a valid expression. Thanks, Jinfeng Ni
