Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/938#discussion_r137939496
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/common/HashTable.java
---
@@ -58,7 +59,7 @@
public int getHashCode(int incomingRowIdx) throws SchemaChangeException;
- public PutStatus put(int incomingRowIdx, IndexPointer htIdxHolder, int
hashCode) throws SchemaChangeException;
+ public PutStatus put(int incomingRowIdx, IndexPointer htIdxHolder, int
hashCode) throws SchemaChangeException, RetryAfterSpillException;
--- End diff --
At present, `RetryAfterSpillException` is unchecked, so it is not necessary
to declare. But, change `RetryAfterSpillException` to extend `Exception`
(checked) and this declaration then becomes useful.
---
