Liran Zelkha has uploaded a new change for review.

Change subject: engine: Fix issue with thread naming
......................................................................

engine: Fix issue with thread naming

Patch http://gerrit.ovirt.org/#/c/19744/ appended org.ovirt.thread
to the thread name. However, this happened again and again, with
no limit. This patch fixes it.

Change-Id: I4094f70859802a11cbd9150461e5af1ae583a17a
Signed-off-by: lzel...@redhat.com <lzel...@redhat.com>
---
M 
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/threadpool/ThreadPoolUtil.java
1 file changed, 4 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/16/20916/1

diff --git 
a/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/threadpool/ThreadPoolUtil.java
 
b/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/threadpool/ThreadPoolUtil.java
index 4f81ba5..c1d259c 100644
--- 
a/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/threadpool/ThreadPoolUtil.java
+++ 
b/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/threadpool/ThreadPoolUtil.java
@@ -45,7 +45,10 @@
         @Override
         protected void beforeExecute(Thread t, Runnable r) {
             super.beforeExecute(t, r);
-            t.setName("org.ovirt.thread." + t.getName());
+            String threadName = t.getName();
+            if (!threadName.startsWith("org.ovirt.thread.")) {
+                t.setName("org.ovirt.thread." + threadName);
+            }
             if (log.isDebugEnabled()) {
                 log.debug("About to run task " + r.getClass().getName() + " 
from ", new Exception());
             }


-- 
To view, visit http://gerrit.ovirt.org/20916
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4094f70859802a11cbd9150461e5af1ae583a17a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liran Zelkha <lzel...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to