Change in ovirt-engine[master]: core: MultipleStorageDomainsValidator#ValidatorPredicate

Code Review gerrit at ovirt.org
Tue Dec 27 20:17:40 UTC 2016


>From Allon Mureinik <amureini at redhat.com>:

Allon Mureinik has submitted this change and it was merged.

Change subject: core: MultipleStorageDomainsValidator#ValidatorPredicate
......................................................................


core: MultipleStorageDomainsValidator#ValidatorPredicate

The ValidatorPredicate is no more than an over-glorified
java.util.Function, and the only reason it existed is that it was
written (well) before oVirt Engine migrated to Java 8.

This patch removes that useless interface, and replaces it with the
standard java.util.Function.

Change-Id: Ibc717a7709e138a166e5e392c642d6e8b780860f
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/MultipleStorageDomainsValidator.java
1 file changed, 26 insertions(+), 50 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Allon Mureinik: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibc717a7709e138a166e5e392c642d6e8b780860f
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list