[Django] #30759: Make cache.delete() return whether or not it suceeded.

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

[Django] #30759: Make cache.delete() return whether or not it suceeded.

Django
#30759: Make cache.delete() return whether or not it suceeded.
-----------------------------------------------+------------------------
               Reporter:  Simon Charette       |          Owner:  nobody
                   Type:  New feature          |         Status:  new
              Component:  Core (Cache system)  |        Version:  2.2
               Severity:  Normal               |       Keywords:
           Triage Stage:  Unreviewed           |      Has patch:  0
    Needs documentation:  0                    |    Needs tests:  0
Patch needs improvement:  0                    |  Easy pickings:  0
                  UI/UX:  0                    |
-----------------------------------------------+------------------------
 It can be quite useful when dealing with complex caching/locking systems
 or simply for logging purposes.

 Memcache clients already returns this value and it should be
 straigtforward to implement for file, inmemory, and database backend based
 on the number of returned rows.

 Redis `del` operation also returns the number of keys it successfully
 deleted so it should be implementable if it's eventually added as
 discussed on the mailing list.

--
Ticket URL: <https://code.djangoproject.com/ticket/30759>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/052.b10be96aac8065a8dcb0ca10e9fbda44%40djangoproject.com.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #30759: Make cache.delete() return whether or not it suceeded.

Django
#30759: Make cache.delete() return whether or not it suceeded.
-------------------------------------+------------------------------------
     Reporter:  Simon Charette       |                    Owner:  nobody
         Type:  New feature          |                   Status:  new
    Component:  Core (Cache system)  |                  Version:  2.2
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+------------------------------------
Changes (by Carlton Gibson):

 * stage:  Unreviewed => Accepted


Comment:

 Seems fair yes. (An individual backend could note that this wasn't
 supported, if that were necessary.)

--
Ticket URL: <https://code.djangoproject.com/ticket/30759#comment:1>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.f83a9c741370c1bb1da69ea8a0766c5a%40djangoproject.com.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #30759: Make cache.delete() return whether or not it suceeded.

Django
In reply to this post by Django
#30759: Make cache.delete() return whether or not it suceeded.
-------------------------------------+-------------------------------------
     Reporter:  Simon Charette       |                    Owner:  Daniel
                                     |  Rios
         Type:  New feature          |                   Status:  assigned
    Component:  Core (Cache system)  |                  Version:  2.2
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Daniel Rios):

 * owner:  nobody => Daniel Rios
 * status:  new => assigned


--
Ticket URL: <https://code.djangoproject.com/ticket/30759#comment:2>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.704b8f8885c160063bf4c4cd3cf5d7f7%40djangoproject.com.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #30759: Make cache.delete() return whether or not it suceeded.

Django
In reply to this post by Django
#30759: Make cache.delete() return whether or not it suceeded.
-------------------------------------+-------------------------------------
     Reporter:  Simon Charette       |                    Owner:  Daniel
                                     |  Rios
         Type:  New feature          |                   Status:  assigned
    Component:  Core (Cache system)  |                  Version:  2.2
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Daniel Rios):

 * has_patch:  0 => 1


--
Ticket URL: <https://code.djangoproject.com/ticket/30759#comment:3>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.963f07efbfd5e4b90819fd1fbe95c95a%40djangoproject.com.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #30759: Make cache.delete() return whether or not it suceeded.

Django
In reply to this post by Django
#30759: Make cache.delete() return whether or not it suceeded.
-------------------------------------+-------------------------------------
     Reporter:  Simon Charette       |                    Owner:  Daniel
                                     |  Rios
         Type:  New feature          |                   Status:  assigned
    Component:  Core (Cache system)  |                  Version:  2.2
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  1
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Simon Charette):

 * needs_better_patch:  0 => 1


Comment:

 Left some comments for improvements on the PR regarding `delete_many` and
 the return type of `delete`.

--
Ticket URL: <https://code.djangoproject.com/ticket/30759#comment:4>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.124032ded8ef0330de42db77ad758fca%40djangoproject.com.