Maor Lipchuk has posted comments on this change.

Change subject: backend: don't cache isoPrefix when empty
......................................................................


Patch Set 1: Code-Review+1

Acttually, the problem which the patch solves here is much more worst.
Empty prefix will be set in the cache map every time the DC will not have ISO 
domain attached to it.
If user will attach an ISO domain after the cache map has been already 
initialized with the empty string, then the user will not be able to attach CD 
to its VMs

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6f77f92c58d9e9336e15302809195928a1ac84a6
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimo...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to