Change in vdsm[master]: New connection management API

ilvovsky at redhat.com ilvovsky at redhat.com
Wed Feb 15 15:11:26 UTC 2012


Igor Lvovsky has posted comments on this change.

Change subject: New connection management API
......................................................................


Patch Set 4: (1 inline comment)

Partially reviewed.

Maybe we can pass to storageServer_ConnectionRefs_statuses()
single connection refID to get status for this connection only.

....................................................
File vdsm/storage/storageServer.py
Line 563:                 for conId, con in self._conDict.items():
why this unnecessary copy?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: If2c15799a83e7b3f39d12c2aa7c91f321156e663
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list