From Tal Nisan <tnisan@redhat.com>:

Tal Nisan has submitted this change and it was merged. Change subject: backend: refactor SyncLunsInfoForBlockStorageDomainCommand ...................................................................... backend: refactor SyncLunsInfoForBlockStorageDomainCommand The objects updateExistingLuns, saveNewLuns and noOp are consumers that update/save/do-nothing given a list of luns, and are a part of the syncing process of storage domains' luns. This patch changes the way we work with these objects. Instead of checking their existence in the map that we get from getLunsToUpdateInDb, each object now contains the relevant info about itself - requiresDbUpdate and affectsDiscardFunctionality. This way the process is more general since the interface is decoupled from the implementation. Change-Id: I264543c2a9500c51c0263f4fc199b117b2103430 Related-to: https://bugzilla.redhat.com/1439692 Signed-off-by: Idan Shaby <ishaby@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/SyncLunsInfoForBlockStorageDomainCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/domain/SyncLunsInfoForBlockStorageDomainCommandTest.java 2 files changed, 74 insertions(+), 31 deletions(-) Approvals: Tal Nisan: Passed CI tests Allon Mureinik: Looks good to me, approved Idan Shaby: Verified Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/77073 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I264543c2a9500c51c0263f4fc199b117b2103430 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Idan Shaby <ishaby@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland@redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>