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 <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Arik Hadas <aha...@redhat.com> Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <mskri...@redhat.com> Gerrit-Reviewer: Moti Asayag <masa...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Roy Golan <rgo...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
