-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33651/
-----------------------------------------------------------
Review request for drill, Mehant Baid and Parth Chandra.
Bugs: drill-2884
https://issues.apache.org/jira/browse/drill-2884
Repository: drill-git
Description
-------
Changed DrillResultSetImpl to throw new ExecutionCanceledSqlException, rather
than (previously existing) AlreadyClosedSqlException, at the first call to the
result set after the associated SQL statement's execution is canceled (except
that currently only some methods check and throw those exceptions).
Also fixed two little documentation errors in DrillResultSet.
Diffs
-----
exec/jdbc/src/main/java/org/apache/drill/jdbc/DrillResultSet.java 7a4f426
exec/jdbc/src/main/java/org/apache/drill/jdbc/DrillStatement.java ba265e6
exec/jdbc/src/main/java/org/apache/drill/jdbc/ExecutionCanceledSqlException.java
PRE-CREATION
exec/jdbc/src/main/java/org/apache/drill/jdbc/impl/DrillResultSetImpl.java
484a5e5
Diff: https://reviews.apache.org/r/33651/diff/
Testing
-------
Manually tested in SQLLine, straight and in debugger (to have control-C INT
thread run at different times relative to query execution flow).
Ran regular tests.
Thanks,
Daniel Barclay
