johnny-cache icon indicating copy to clipboard operation
johnny-cache copied to clipboard
verified publisher icon jmoiron

Added support to disable caching for random (ORDER BY RAND()) queries

Open emre opened this issue 13 years ago • 5 comments

emre avatar Jul 24 '12 19:07 emre

+1

kevinastone avatar Feb 06 '13 04:02 kevinastone

@jmoiron, could you look at this?

random queries should not be cached.

emre avatar Feb 06 '13 08:02 emre

Yeah, just to expand on my +1, I got bit by this and was forced to switch to @emre 's fork (thanks btw) to continue using Johnny Cache.

What's the hold up in merging this change?

kevinastone avatar Feb 06 '13 20:02 kevinastone

Thanks for the submission, most likely we'll be going through all of the bugs again with django 1.5 (and since it involves a bunch of queryset changes, we may have to change a lot). It looks fine, we'll just need to write a test for it and run the full test suite for the versions we'll be supporting in 1.5.

jself avatar Feb 06 '13 21:02 jself

Any update on this one? 1.5 has been out for quite awhile now.

If you need help writing tests for things can you please elaborate on what you would like tested and I'd be happy to help write some tests.

borgstrom avatar Mar 20 '13 21:03 borgstrom