Allon Mureinik has submitted this change and it was merged. ( https://gerrit.ovirt.org/80313 ) Change subject: core: StorageServerConnectionDaoTest#existingNfsAutoConnection ...................................................................... core: StorageServerConnectionDaoTest#existingNfsAutoConnection The only usage for this member is its ID, which can be taken from FixturesTool anyway. This patch removes this redundant member, both in order to clean up the code and to speed up the test by removing a useless DAO call from the test's setup. Change-Id: Iaa95d1ec41d9cf55bbbfeb367b6e76a4d5d52dc1 Signed-off-by: Allon Mureinik <amureini(a)redhat.com&gt; --- M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/StorageServerConnectionDaoTest.java 1 file changed, 1 insertion(+), 3 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/80313 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaa95d1ec41d9cf55bbbfeb367b6e76a4d5d52dc1 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com&gt; Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com&gt; Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com&gt; Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;