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(a)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(a)redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>