Change in ovirt-engine[master]: core: Remove SyntaxChecker searchResultLimits

Code Review gerrit at ovirt.org
Mon May 15 15:09:24 UTC 2017


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

Allon Mureinik has submitted this change and it was merged.

Change subject: core: Remove SyntaxChecker searchResultLimits
......................................................................


core: Remove SyntaxChecker searchResultLimits

Removed the unused constructor parameter.

Note that removing this parameter means that the SyntaxCheckerFactory
class no longer relies on ConfigValues, and the tests can be cleaned
up.

Change-Id: I9ad3d75860e1fdcbf8b97e8e2c73ca2491c356c2
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SyntaxChecker.java
M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SyntaxCheckerFactory.java
M backend/manager/modules/searchbackend/src/test/java/org/ovirt/engine/core/searchbackend/SyntaxCheckerFactoryTest.java
M backend/manager/modules/searchbackend/src/test/java/org/ovirt/engine/core/searchbackend/SyntaxCheckerTest.java
4 files changed, 7 insertions(+), 17 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Allon Mureinik: Verified
  Moti Asayag: Looks good to me, approved
  Eli Mesika: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9ad3d75860e1fdcbf8b97e8e2c73ca2491c356c2
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: Eli Mesika
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list