Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2571
to look at the new patch set (#4).
Change subject: Fix Spark shutdown
......................................................................
Fix Spark shutdown
The Kudu client was creating Netty timers with non-daemon threads, which
prevented the Spark shell JVM from beginning the shutdown process. This commit
also adds a shutdown hook to the Spark connector so that when the shell process
exits it will properly shut down the Kudu client which prevents batched but
unflushed writes from being lost.
See
http://apache-spark-developers-list.1001551.n3.nabble.com/graceful-shutdown-in-external-data-sources-td16684.html
for more discussion.
Change-Id: I30a64ec5eb30d70361204646523c9947d88c251f
---
M java/kudu-client/src/main/java/org/kududb/client/AsyncKuduClient.java
M java/kudu-spark/src/main/scala/org/kududb/spark/KuduContext.scala
2 files changed, 38 insertions(+), 5 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/71/2571/4
--
To view, visit http://gerrit.cloudera.org:8080/2571
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I30a64ec5eb30d70361204646523c9947d88c251f
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Binglin Chang <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
