#36732: Sitemaps with i18n=True load the whole table in memory
-------------------------------------+-------------------------------------
     Reporter:  Julien Palard        |                    Owner:  (none)
         Type:                       |                   Status:  new
  Cleanup/optimization               |
    Component:  contrib.sitemaps     |                  Version:  5.2
     Severity:  Normal               |               Resolution:
     Keywords:  sitemap, memory      |             Triage Stage:
                                     |  Unreviewed
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Comment (by Julien Palard):

 While:

 {{{
     def get_languages_for_item(self, item):
         """Languages for which this item is displayed."""
         return self._languages()
 }}}



 does not use item, something (non trivial) could be implemented to slice
 the queryset at the right spot (depending on the number of languages), and
 the total number of elements could be computed by a simple multiplication.

 But as long as get_languages_for_item uses item all of this break.
-- 
Ticket URL: <https://code.djangoproject.com/ticket/36732#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 [email protected].
To view this discussion visit 
https://groups.google.com/d/msgid/django-updates/0107019a82a613ca-65bf05c9-f340-4396-b1bd-10124b80f87a-000000%40eu-central-1.amazonses.com.

Reply via email to