Nir Soffer has posted comments on this change.

Change subject: storage: _upgradePool - concurrent.thread instead 
threading.Thread
......................................................................


Patch Set 2:

Coping from lib/vdsm/concurrent.py:

def thread(func, args=(), kwargs=None, name=None, daemon=True, logger=None):
    """ 
    Create a thread for runnning func with args.

    Arguments:

    func        Function to run in a new thread.

    args        Arguments to pass to func

    kwargs      Keyword arguments to pass to func

    name        If set, set thread name.

    daemon      If True, create a daemon thread.

    logger      If set, unhandled exception will be logged on this logger.
                Otherwise the root logger will be used.
    """

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1bb080c25e3dbf3c93f84ce73417e7e64bcb4fc4
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <lara...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <lara...@redhat.com>
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to