Change in ovirt-engine[master]: core: remove vm usage related histories from the database

From Arik Hadas <ahadas@redhat.com>:
Arik Hadas has submitted this change and it was merged. Change subject: core: remove vm usage related histories from the database ...................................................................... core: remove vm usage related histories from the database Now that VmStatistics are cached in VmManager, there is no need to dump several things they contain to the database. However, since we currently focus on easy and non-risky things, this patch addresses specifically the memory/cpu/network usage histories - these are really redundant being the database as they are only used by the webadmin that gathers the information using a (search) query that takes this information from VmManager. Unfortunately, we can neither drop the whole statistics from the database nor reduce the frequency of the updates at this stage. This kind of changes can improve performance on large-scale setups significantely since they reduce the amount of data dump into the database substantially. Change-Id: If72063205907c21d9bd80610a1c1ea1356a9c5e6 Signed-off-by: Arik Hadas <ahadas@redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmStatisticsDaoImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmStatisticsDaoTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml M packaging/dbscripts/create_views.sql A packaging/dbscripts/upgrade/04_01_0390_remove_vm_related_usage_histories.sql M packaging/dbscripts/vms_sp.sql 6 files changed, 3 insertions(+), 78 deletions(-) Approvals: Jenkins CI: Passed CI tests Arik Hadas: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/65787 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If72063205907c21d9bd80610a1c1ea1356a9c5e6 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Eli Mesika <emesika@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: Martin Sivák <msivak@redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek@redhat.com> Gerrit-Reviewer: Roy Golan <rgolan@redhat.com> Gerrit-Reviewer: Shirly Radco <sradco@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek@redhat.com> Gerrit-Reviewer: Yaniv Kaul <ykaul@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>
participants (1)
-
Code Review