Sort everything by K

Bug #1206153 reported by Flavio Percoco
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
Fix Released
High
Flavio Percoco

Bug Description

We used to use _id to guarantee FIFO, since we now have k, we should make sure we use it to sort all queries that have to be sorted.

This should make some indexes smaller.

Changed in marconi:
importance: Undecided → High
assignee: nobody → Flavio Percoco (flaper87)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to marconi (master)

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

Changed in marconi:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to marconi (master)

Reviewed: https://review.openstack.org/39457
Committed: http://github.com/stackforge/marconi/commit/70a3ab6ba17d05ed8db1bee0ca6a12337de936c8
Submitter: Jenkins
Branch: master

commit 70a3ab6ba17d05ed8db1bee0ca6a12337de936c8
Author: Flaper Fesp <email address hidden>
Date: Wed Jul 31 13:15:38 2013 +0200

    Sort everything by K

    The 'k' field is the one we're using to guarantee FIFO. Instead of
    sorting by _id, which we were using for the same purpose, the message
    controller should always use 'k' for that matter.

    Fixes bug: #1206153

    Change-Id: I74704ddb50de35e3d50c4108b28d063376a34cf8

Changed in marconi:
status: In Progress → Fix Committed
Changed in marconi:
milestone: none → havana-3
status: Fix Committed → Fix Released
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.