no any index for port DB sqlalchemy

[shihanzhang]

Now there is not any index for port DB sqlalchemy, but for nova bulk create VM, it frequently select port according to 'tenant_id','network_id' ,'device_id', if no index, it will consume much time in db operation, so I think it is better to add index for
port 'tenant_id', 'network_id' ,'device_id'.

'tenant_id' is added in https://bugs.launchpad.net/neutron/+bug/1419815.

[Ann Taraday (akamyshnikova)]

Index for tenant_id is already added in https://bugs.launchpad.net/neutron/+bug/1419815.

[Ann Taraday (akamyshnikova)]

Also I suggest to provide some examples that adding index really will speed up requests.

[shihanzhang (shihanzhang)]

hi Ann Kamyshnikova, thanks for your response, I will add examples and remove 'tenant_id' in my patch!

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/169641

[Henry Gessau (gessau)]

See attached image.

blue line means: no index
read line means: tenant_id + device_id
green line means: tenant_id + device_id + network_id + mac_address
purple line means: tenant_id + device_id + network_id + mac_address + device_owner

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/169641
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=f93007952e40a7ae7cb2d4f3588059acf42ea209
Submitter: Jenkins
Branch: master

commit f93007952e40a7ae7cb2d4f3588059acf42ea209
Author: shihanzhang <email address hidden>
Date: Tue Mar 31 16:14:12 2015 +0800

    Add index for port

    This patch will speed up SELECTs Port with filters by
    'network_id + device_owner' and 'network_id + mac_address'.

    Closes-bug: #1421089

    Change-Id: Ied90b6304df971a6049871f65df3e1aaee624647

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: neutron-pecan
Review: https://review.openstack.org/185072