Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 9 additions & 9 deletions manila/db/sqlalchemy/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -590,7 +590,7 @@ def state(self):

instance_mappings = orm.relationship(
"ShareInstanceAccessMapping",
lazy='immediate',
lazy='selectin',
primaryjoin=(
'and_('
'ShareAccessMapping.id == '
Expand Down Expand Up @@ -621,7 +621,7 @@ class ShareAccessRulesMetadata(BASE, ManilaBase):
access = orm.relationship(
ShareAccessMapping, backref="share_access_rules_metadata",
foreign_keys=access_id,
lazy='immediate',
lazy='selectin',
primaryjoin='and_('
'ShareAccessRulesMetadata.access_id == ShareAccessMapping.id,'
'ShareAccessRulesMetadata.deleted == "False")')
Expand All @@ -646,7 +646,7 @@ def set_share_access_data(self, share_access):

instance = orm.relationship(
"ShareInstance",
lazy='immediate',
lazy='selectin',
primaryjoin=(
'and_('
'ShareInstanceAccessMapping.share_instance_id == '
Expand Down Expand Up @@ -856,7 +856,7 @@ def size(self):

export_locations = orm.relationship(
"ShareSnapshotInstanceExportLocation",
lazy='immediate',
lazy='selectin',
primaryjoin=(
'and_('
'ShareSnapshotInstance.id == '
Expand All @@ -866,15 +866,15 @@ def size(self):
)
share_instance = orm.relationship(
ShareInstance, backref="snapshot_instances",
lazy='immediate',
lazy='selectin',
primaryjoin=(
'and_('
'ShareSnapshotInstance.share_instance_id == ShareInstance.id,'
'ShareSnapshotInstance.deleted == "False")')
)
snapshot = orm.relationship(
"ShareSnapshot",
lazy="immediate",
lazy='selectin',
foreign_keys=snapshot_id,
backref="instances",
primaryjoin=(
Expand All @@ -888,7 +888,7 @@ def size(self):
)
share_group_snapshot = orm.relationship(
"ShareGroupSnapshot",
lazy="immediate",
lazy='selectin',
foreign_keys=share_group_snapshot_id,
backref="share_group_snapshot_members",
primaryjoin=('ShareGroupSnapshot.id == '
Expand Down Expand Up @@ -921,7 +921,7 @@ def state(self):

instance_mappings = orm.relationship(
"ShareSnapshotInstanceAccessMapping",
lazy='immediate',
lazy='selectin',
primaryjoin=(
'and_('
'ShareSnapshotAccessMapping.id == '
Expand Down Expand Up @@ -952,7 +952,7 @@ def set_snapshot_access_data(self, snapshot_access):

instance = orm.relationship(
"ShareSnapshotInstance",
lazy='immediate',
lazy='selectin',
primaryjoin=(
'and_('
'ShareSnapshotInstanceAccessMapping.share_snapshot_instance_id == '
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
---
fixes:
- |
Improved share snapshot and access list speed using lazy='selectin'. The
sqlalchemy models of share snapshot and access rule relationships previously
had lazy='immediate'. This resulted in extra queries when querying for all
share snapshot and access rule details.
Loading