The Nonce model needs an index

Bug #1842043 reported by Alberto Donato
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
django-piston3 (Ubuntu)
Won't Fix
Low
Unassigned

Bug Description

The Nonce model needs a (ideally unique) index on (token_key, consumer_key, key).

Aside from granting there to be only one entry for each combination, the index will speed up search for nonces when API calls are performed.

Given Nonce entries are never deleted, the table can grow quite big, and doing a full scan to find if there's a matching entry becomes increasingly more costly.

Tags: patch

Related branches

Changed in django-piston3 (Ubuntu):
status: New → In Progress
Revision history for this message
Alberto Donato (ack) wrote :

The patch adds the index to the Nonce model

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "diff.diff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

tags: added: patch
Mathew Hodson (mhodson)
Changed in django-piston3 (Ubuntu):
importance: Undecided → Low
Revision history for this message
Simon Quigley (tsimonq2) wrote :

Hi Alberto! Apologies for the delay in reviewing your patch.

According to LP #1882949, it seems this package has been removed from Groovy. If you feel this should be SRU'ed, please update the bug description to follow the Stable Release Update guidelines, which can be found here: https://wiki.ubuntu.com/StableReleaseUpdates#SRU_Bug_Template

I have unsubscribed ~ubuntu-sponsors, as there is nothing left to do here. Please resubscribe us if you would like to get this in a stable release.

Revision history for this message
Alberto Donato (ack) wrote :

We don't really need this fix SRU'd. maas carries a database patch for the index, and as mentioned the package has been dropped from the archive (and maas has a fixed version in PPAs).

Thanks.

Simon Quigley (tsimonq2)
Changed in django-piston3 (Ubuntu):
status: In Progress → Won't Fix
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.