Yaniv Bronhaim has posted comments on this change.
Change subject: Changing threadName when using multiprocessing package ......................................................................
Patch Set 1:
Only for functions that use the logDecorator (almost all svdsm api), and it won't run every log print, but every svdsm call before we run the function.
I don't think it's expensive here, print the log is much more complex then that. To calculate thread name before running the function is reasonable
-- 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 ybronhei@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Douglas Schilling Landgraf dougsland@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Shu Ming shuming@linux.vnet.ibm.com Gerrit-Reviewer: Yaniv Bronhaim ybronhei@redhat.com Gerrit-Reviewer: Yeela Kaplan ykaplan@redhat.com