Arik Hadas has submitted this change and it was merged. Change subject: core: Minor refactoring of AttachUserToVmFromPoolAndRunCommand ...................................................................... core: Minor refactoring of AttachUserToVmFromPoolAndRunCommand * Removed redundant poolId parameters; * Removed redundant DbFacade references; * Removed redundant get/setVmPoolId() override; * Fixed formatting. Change-Id: Ife028ba761ae965ae0d49b254724795ff4a2245f Signed-off-by: Shmuel Melamud <smelamud@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AttachUserToVmFromPoolAndRunCommand.java 1 file changed, 29 insertions(+), 36 deletions(-) Approvals: Jenkins CI: Passed CI tests Shmuel Leib Melamud: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/57829 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ife028ba761ae965ae0d49b254724795ff4a2245f Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shmuel Leib Melamud <smelamud@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shmuel Leib Melamud <smelamud@redhat.com> Gerrit-Reviewer: Shmuel Melamud <smelamud@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>