Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/934#discussion_r137941417
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/fragment/FragmentStatusReporter.java
---
@@ -113,4 +120,14 @@ void fail(final UserException ex) {
sendStatus(status);
}
+ @Override
+ public void close()
+ {
+ final ControlTunnel tunnel = this.tunnel.getAndSet(null);
+ if (tunnel != null) {
+ logger.debug("Closing {}", this);
--- End diff --
No, close() is not a placeholder. It closes FragmentStatusReporter and
after the close, request to send status becomes no-op.
---
