Change in ovirt-engine[ovirt-engine-4.1]: async-command: Remove setting parent command

Code Review gerrit at ovirt.org
Wed May 10 15:00:22 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged.

Change subject: async-command: Remove setting parent command
......................................................................


async-command: Remove setting parent command

If setting the RunAsyncActionCommand as the parent of the command being
executed (original command), the original command "endAction" method
isn't invoked, leaving it not completed.
This change is valid for commands that could be run asynchronously.

Change-Id: Iae4abd65206e4422f1758d8a258c8d36d265f192
Bug-Url: https://bugzilla.redhat.com/1447050
Signed-off-by: Ala Hino <ahino at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunAsyncActionCommand.java
1 file changed, 2 insertions(+), 8 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, approved
  Ala Hino: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iae4abd65206e4422f1758d8a258c8d36d265f192
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list