/juju

Add index for migration of status history values. #7059

Merged
merged 1 commit into from Mar 2, 2017
+2 −0

Conversation

Reviewers

@babbageclunk babbageclunk

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@howbazaar @jujubot @babbageclunk
@howbazaar
Owner

howbazaar commented Mar 2, 2017

Description of change

In larger controllers, the statuses history collection gets too large for simple sorting queries.

QA steps

Tested with blahdeblah interactively in a 2.1 branch.

Bug reference

https://bugs.launchpad.net/juju/+bug/1668646
@babbageclunk

babbageclunk approved these changes Mar 2, 2017
View changes

Would that all changes were so easy to review.

@howbazaar
Owner

howbazaar commented Mar 2, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Mar 2, 2017

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit 2bbf576 into juju:2.1 Mar 2, 2017

@howbazaar howbazaar deleted the howbazaar:statuseshistory-index branch Mar 2, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment