Francesco Romani has uploaded a new change for review.

Change subject: jsonrpc: shorten executor thread' system  names
......................................................................

jsonrpc: shorten executor thread' system  names

We shorten the basename of the jsonrpc executor,
because the system has a hard limit of 15 ASCII
character per thread name.

The new name is less expressive, but leaves room
for the worker ID, which is an overall usability
improvement.

Before, it looked like

  jsonrpc.Executo

now it looks like

  jsonrpcex/0

Change-Id: Ibefc7d2c60d7b5d23739b129088f8e7f80514540
Signed-off-by: Francesco Romani <from...@redhat.com>
---
M lib/vdsm/rpc/bindingjsonrpc.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/78/61978/1

diff --git a/lib/vdsm/rpc/bindingjsonrpc.py b/lib/vdsm/rpc/bindingjsonrpc.py
index 1f6f09a..1e55672 100644
--- a/lib/vdsm/rpc/bindingjsonrpc.py
+++ b/lib/vdsm/rpc/bindingjsonrpc.py
@@ -35,7 +35,7 @@
     log = logging.getLogger('BindingJsonRpc')
 
     def __init__(self, bridge, subs, timeout, scheduler, cif):
-        self._executor = executor.Executor(name="jsonrpc.Executor",
+        self._executor = executor.Executor(name="jsonrpcex",
                                            workers_count=_THREADS,
                                            max_tasks=_TASKS,
                                            scheduler=scheduler)


-- 
To view, visit https://gerrit.ovirt.org/61978
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibefc7d2c60d7b5d23739b129088f8e7f80514540
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <from...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Reply via email to