Vojtech Szocs has posted comments on this change.

Change subject: webadmin: Fix UI plugin REST API HttpSession refresh issue
......................................................................


Patch Set 1:

> Or if we should have the OperationManager simply fire an event, and have the 
> RestApiRefreshManager listen to that event.

I had this idea but eventually decided not to "pollute" EventBus with very 
frequent "operation executed" events. If you think that using EventBus for 
loose coupling is better, I'll go with that.

-- 
To view, visit http://gerrit.ovirt.org/36622
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2ffd3198239040f67313238a837af98db16e79ea
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vojtech Szocs <vsz...@redhat.com>
Gerrit-Reviewer: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to