Fetching all existing backup entries unnecessarily for incremental backup could be avoidable.

Bug #1696715 reported by Pranali Deore
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Cinder
In Progress
Undecided
Sean McGinnis

Bug Description

In create method of backup, while doing an incremental backup all existing backup entries gets fetched from DB to search for the one to use in incremental.

IMO, only required one could be fetched by sorting the the result in right order during the DB call instead of fetching all and picking up the one later.

Changed in cinder:
assignee: nobody → Pranali Deore (pranali-deore)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
status: New → In Progress
Changed in cinder:
assignee: Pranali Deore (pranali-deore) → Sean McGinnis (sean-mcginnis)
Revision history for this message
Christian Rohmann (christian-rohmann) wrote :

I rebased and reworked the patch a little. If someone cares to take a look. Would be a nice improvement.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.