Hello Jean-Daniel Cryans, Todd Lipcon,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/2571

to review the following change.

Change subject: Fix Spark shutdown
......................................................................

Fix Spark shutdown

The Kudu client was creating Netty timers with non-daemon threads, which led to
the Spark shell being unable to close properly. This commit also adds a shutdown
hook to the Spark connector so that when the shell processes exits it will
properly shut down the Kudu client.

Change-Id: I30a64ec5eb30d70361204646523c9947d88c251f
---
M java/kudu-client/src/main/java/org/kududb/client/AsyncKuduClient.java
M java/kudu-client/src/main/java/org/kududb/client/KuduRpcResponse.java
M java/kudu-spark/src/main/scala/org/kududb/spark/KuduContext.scala
3 files changed, 25 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/71/2571/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2571
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I30a64ec5eb30d70361204646523c9947d88c251f
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to