Moti Asayag has submitted this change and it was merged. Change subject: core: AAAServiceImpl#dumpRecord ignore param ...................................................................... core: AAAServiceImpl#dumpRecord ignore param The ignore parameter of dumpRecord contains a series of keys to be ignored when dumping the record. Since the only method used there on it is contains, using a Set makes more sense than using a List. Change-Id: Ib1b330bea3228afc0e7c2196c117595bd2f35c95 Signed-off-by: Allon Mureinik <amureini(a)redhat.com&gt; --- M backend/manager/extensions-tool/src/main/java/org/ovirt/engine/exttool/aaa/AAAServiceImpl.java 1 file changed, 5 insertions(+), 6 deletions(-) Approvals: Ondra Machacek: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Allon Mureinik: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/74644 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib1b330bea3228afc0e7c2196c117595bd2f35c95 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: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com&gt; Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com&gt; Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com&gt; Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com&gt; Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;