You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@giraph.apache.org by ma...@apache.org on 2014/08/01 20:47:30 UTC

git commit: updated refs/heads/trunk to 6d4af60

Repository: giraph
Updated Branches:
  refs/heads/trunk ce97134d2 -> 6d4af60e7


GIRAPH-936: AsyncMessageStoreWrapper threads are not daemonized (edunov via majakabiljo)


Project: http://git-wip-us.apache.org/repos/asf/giraph/repo
Commit: http://git-wip-us.apache.org/repos/asf/giraph/commit/6d4af60e
Tree: http://git-wip-us.apache.org/repos/asf/giraph/tree/6d4af60e
Diff: http://git-wip-us.apache.org/repos/asf/giraph/diff/6d4af60e

Branch: refs/heads/trunk
Commit: 6d4af60e729a6b2153c220046cf6a3804427f1c1
Parents: ce97134
Author: Maja Kabiljo <ma...@fb.com>
Authored: Fri Aug 1 11:46:52 2014 -0700
Committer: Maja Kabiljo <ma...@fb.com>
Committed: Fri Aug 1 11:46:52 2014 -0700

----------------------------------------------------------------------
 CHANGELOG                                                          | 2 ++
 .../giraph/comm/messages/queue/AsyncMessageStoreWrapper.java       | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/giraph/blob/6d4af60e/CHANGELOG
----------------------------------------------------------------------
diff --git a/CHANGELOG b/CHANGELOG
index dbb134a..3c8e155 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,6 +1,8 @@
 Giraph Change Log
 
 Release 1.1.0 - unreleased
+  GIRAPH-936: AsyncMessageStoreWrapper threads are not daemonized (edunov via majakabiljo)
+
   GIRAPH-934: Allow having state in aggregators (ikabiljo via majakabiljo)
 
   GIRAPH-932: Adding .arcconfig to GIRAPH for Arcanist support (aching)

http://git-wip-us.apache.org/repos/asf/giraph/blob/6d4af60e/giraph-core/src/main/java/org/apache/giraph/comm/messages/queue/AsyncMessageStoreWrapper.java
----------------------------------------------------------------------
diff --git a/giraph-core/src/main/java/org/apache/giraph/comm/messages/queue/AsyncMessageStoreWrapper.java b/giraph-core/src/main/java/org/apache/giraph/comm/messages/queue/AsyncMessageStoreWrapper.java
index a62834f..252ee39 100644
--- a/giraph-core/src/main/java/org/apache/giraph/comm/messages/queue/AsyncMessageStoreWrapper.java
+++ b/giraph-core/src/main/java/org/apache/giraph/comm/messages/queue/AsyncMessageStoreWrapper.java
@@ -60,7 +60,7 @@ public final class AsyncMessageStoreWrapper<I extends WritableComparable,
   /** Executor that processes messages in background */
   private static final ExecutorService EXECUTOR_SERVICE =
       Executors.newCachedThreadPool(
-          new ThreadFactoryBuilder()
+          new ThreadFactoryBuilder().setDaemon(true)
               .setNameFormat("AsyncMessageStoreWrapper-%d").build());
 
   /** Number of threads that will process messages in background */