Re: [Django] #33046: Fully cached result approach of the QuerySet.count is not documented.

2021-08-26 Thread Django
#33046: Fully cached result approach of the QuerySet.count is not documented.
-+-
 Reporter:  Can Sarıgöl  |Owner:  Can
 Type:   |  Sarıgöl
  Cleanup/optimization   |   Status:  closed
Component:  Documentation|  Version:  3.2
 Severity:  Normal   |   Resolution:  fixed
 Keywords:   | Triage Stage:  Ready for
 |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Mariusz Felisiak ):

 In [changeset:"69009f495232dbab7ec02b908fefa67147c76d64" 69009f49]:
 {{{
 #!CommitTicketReference repository=""
 revision="69009f495232dbab7ec02b908fefa67147c76d64"
 [3.2.x] Fixed #33046 -- Added note about using length of cached result by
 QuerySet.count().

 Backport of 7ea86076fc1128d43b47f4826cb25eb5ebb19f19 from main
 }}}

-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.38d6cf1c51dcb1a1cba7add52320cc14%40djangoproject.com.


Re: [Django] #33046: Fully cached result approach of the QuerySet.count is not documented.

2021-08-25 Thread Django
#33046: Fully cached result approach of the QuerySet.count is not documented.
-+-
 Reporter:  Can Sarıgöl  |Owner:  Can
 Type:   |  Sarıgöl
  Cleanup/optimization   |   Status:  closed
Component:  Documentation|  Version:  3.2
 Severity:  Normal   |   Resolution:  fixed
 Keywords:   | Triage Stage:  Ready for
 |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Mariusz Felisiak ):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"7ea86076fc1128d43b47f4826cb25eb5ebb19f19" 7ea8607]:
 {{{
 #!CommitTicketReference repository=""
 revision="7ea86076fc1128d43b47f4826cb25eb5ebb19f19"
 Fixed #33046 -- Added note about using length of cached result by
 QuerySet.count().
 }}}

-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.252820a8df81937d5ba97e5b19c815e5%40djangoproject.com.


Re: [Django] #33046: Fully cached result approach of the QuerySet.count is not documented.

2021-08-25 Thread Django
#33046: Fully cached result approach of the QuerySet.count is not documented.
-+-
 Reporter:  Can Sarıgöl  |Owner:  Can
 Type:   |  Sarıgöl
  Cleanup/optimization   |   Status:  assigned
Component:  Documentation|  Version:  3.2
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Ready for
 |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Mariusz Felisiak):

 * owner:  nobody => Can Sarıgöl
 * status:  new => assigned
 * has_patch:  0 => 1
 * stage:  Accepted => Ready for checkin


Comment:

 [https://github.com/django/django/pull/14799 PR]

-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.173aeb390df3a4ecb4e0ee89238f4544%40djangoproject.com.


Re: [Django] #33046: Fully cached result approach of the QuerySet.count is not documented.

2021-08-24 Thread Django
#33046: Fully cached result approach of the QuerySet.count is not documented.
--+
 Reporter:  Can Sarıgöl   |Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Documentation |  Version:  3.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):

 * type:  Uncategorized => Cleanup/optimization
 * component:  Uncategorized => Documentation
 * stage:  Unreviewed => Accepted


Comment:

 Hey Can! You want to add a note something like, ''If the queryset has
 already been fully retrieved, count will use that length rather than
 perform an additional query'' to the
 
[https://docs.djangoproject.com/en/3.2/ref/models/querysets#django.db.models.query.QuerySet.count
 count() reference]? Sounds OK, yes.

-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.209d2f35bca53534df9f6313c529fa83%40djangoproject.com.


[Django] #33046: Fully cached result approach of the QuerySet.count is not documented.

2021-08-24 Thread Django
#33046: Fully cached result approach of the QuerySet.count is not documented.
-+
   Reporter:  Can Sarıgöl|  Owner:  nobody
   Type:  Uncategorized  | Status:  new
  Component:  Uncategorized  |Version:  3.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  |
-+
 there is no document for count function docstring.


 {{{
 def count(self):
 """
 ...

 If the QuerySet is already fully cached, return the length of the
 cached results set to avoid multiple SELECT COUNT(*) calls.
 """
 }}}

-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/053.ce6b0741c93f0398e55ae37fd38e8b86%40djangoproject.com.