Yaniv Bronhaim has posted comments on this change.

Change subject: Changing threadName when using multiprocessing package
......................................................................


Patch Set 1:

The log format puts the threadName in its head already, the multiprocessing 
implementation modifies the threadName to MainProcess|[threadName] as I 
explained. So it this way, or override the 'connect' function of manager so 
that modification won't process.. in my opinion this is the easiest solution 
and not so bad one.. overriding can be a mess here.. but I can if you prefer

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I541beb63fd5e1a2f7d6eed7f66f4970bf37a6705
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Shu Ming <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Yeela Kaplan <[email protected]>
Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to