Omer Frenkel has posted comments on this change. Change subject: events: process status updates in sequential order ......................................................................
Patch Set 11: we should wait with considering this patch only if we see issues on scaled setups. assuming https://gerrit.ovirt.org/#/c/41520 would be taken in, there is no lost of data, on worst case we might dismiss events that arrive during long getAllStats cycle, which would cause slower update of data. if this will be an issue we need to consider a version of this approach (see comment on code) -- To view, visit https://gerrit.ovirt.org/41589 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id7e165c37246458b5898a5da3c260b80d1cb9cb7 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Moti Asayag <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
