Xiaolei Shi has posted comments on this change.

Change subject: db: Numa support database implementation
......................................................................


Patch Set 21:

(1 comment)

http://gerrit.ovirt.org/#/c/26996/21/packaging/dbscripts/upgrade/03_05_0330_add_numa_tables_and_columns.sql
File packaging/dbscripts/upgrade/03_05_0330_add_numa_tables_and_columns.sql:

Line 52:         vds_numa_node_id UUID,
Line 53:         vds_numa_node_index SMALLINT,
Line 54:         is_pinned BOOLEAN DEFAULT false NOT NULL,
Line 55:         CONSTRAINT pk_vm_vds_numa_node_map PRIMARY KEY(id),
Line 56:         CONSTRAINT fk_vm_vds_numa_node_map_vds_numa_node FOREIGN 
KEY(vds_numa_node_id) REFERENCES numa_node(numa_node_id) ON DELETE SET NULL,
@Gilad, as you said, if vm migrate to another host with the same numa topology, 
we need to update this table to update the vds_numa_node_id according to the 
vds_numa_node_index.
So there should be a process in engine to do the update action, could you give 
me some advice about the code location(in which class or just a submodule) so 
that i can add the action to update the data once vm migrated?
Line 57:         CONSTRAINT fk_vm_vds_numa_node_map_vm_numa_node FOREIGN 
KEY(vm_numa_node_id) REFERENCES numa_node(numa_node_id) ON DELETE CASCADE
Line 58:     );
Line 59: 
Line 60:     -- Create partial index for numa node map


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2507c084aa214bcfb65e860b11ed7dcf02af50cc
Gerrit-PatchSet: 21
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Xiaolei Shi <xiao-lei....@hp.com>
Gerrit-Reviewer: Doron Fediuck <dfedi...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Jason Liao <chuan.l...@hp.com>
Gerrit-Reviewer: Martin Sivák <msi...@redhat.com>
Gerrit-Reviewer: Xiaolei Shi <xiao-lei....@hp.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to