Could you please rewrite this test to use `ExpectedException` rule? It will 
allow avoiding `try catch` block and clearer validating error type. 

Also, I think it is enough to check that error message contains a string 
"VALIDATION ERROR:" instead of checking `getErrorType()` and add a check for a 
string, where current precision and scale are displayed.

[ Full content available at: https://github.com/apache/drill/pull/1451 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to