Change in ovirt-engine[master]: core: Generate UUIDs from engine

emesika at redhat.com emesika at redhat.com
Sat Nov 10 20:33:59 UTC 2012


Eli Mesika has submitted this change and it was merged.

Change subject: core: Generate UUIDs from engine
......................................................................


core: Generate UUIDs from engine

We used to generate UUIDs used as primary keys of bookmarks, tags,
clusters hosts and pools in the database. This patch changes the engine
so that it generates them directly and passes them as parameters to the
stored procedures as needed.

Note that some UUIDs can still be generated using the database function
uuid_generate_v1 during first installation and upgrades, as upgrade
scripts haven't been changed.

Change-Id: Idecd3aa570017b4198243a9d3f9ebfdc7a777195
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M backend/manager/dbscripts/bookmarks_sp.sql
M backend/manager/dbscripts/business_entity_snapshot_sp.sql
M backend/manager/dbscripts/common_sp.sql
M backend/manager/dbscripts/insert_data.sql
M backend/manager/dbscripts/inst_sp.sql
M backend/manager/dbscripts/tags_sp.sql
M backend/manager/dbscripts/vds_groups_sp.sql
M backend/manager/dbscripts/vds_sp.sql
M backend/manager/dbscripts/vm_pools_sp.sql
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/BusinessEntitySnapshot.java
M backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/NGuid.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/BookmarkDAODbFacadeImpl.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/BusinessEntitySnapshotDAODbFacadeImpl.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/TagDAODbFacadeImpl.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsStaticDAODbFacadeImpl.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmPoolDAODbFacadeImpl.java
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/ManageDomainsDAOImpl.java
M packaging/fedora/setup/engine-setup.py
19 files changed, 81 insertions(+), 53 deletions(-)

Approvals:
  Eli Mesika: Verified; Looks good to me, approved
  Juan Hernandez: Verified


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Idecd3aa570017b4198243a9d3f9ebfdc7a777195
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>



More information about the Engine-commits mailing list