Re: Django ORM performance patch. Fixes #5420, #5768

2008-02-16 Thread David Cramer
Per my patches and what I believe Malcom has done with qsrf, the syntax for related fields was field__related__name, and yes, you would repeat this over and over, but it keeps it consistant. I would like a values-like method to return partial objects, and then requesting any field thats not

Re: Django ORM performance patch. Fixes #5420, #5768

2008-02-15 Thread Dima Dogadaylo
On 15 Feb, 12:30, Malcolm Tredinnick <[EMAIL PROTECTED]> wrote: > When Adrian > proposed that API, he realised that almost always you're going to be > pulling back all of the fields or almost all of them. Once a database > has read a row to access some of the data, accessing all of the data in >

Re: Django ORM performance patch. Fixes #5420, #5768

2008-02-15 Thread Sebastian Noack
On Fri, 15 Feb 2008 21:30:23 +1100 Malcolm Tredinnick <[EMAIL PROTECTED]> wrote: > There's also the fairly pragmatic side-effect that if you are only > pulling back a very few fields, there isn't really a lot your model > methods are going to be able to do without loading more data, unless >

Re: Django ORM performance patch. Fixes #5420, #5768

2008-02-15 Thread Sebastian Noack
On Fri, 15 Feb 2008 01:54:24 -0800 (PST) Dima Dogadaylo <[EMAIL PROTECTED]> wrote: > > Of course, this decreases database load, but IMHO it is better to > > use QuerySet.values if you want select just certain values from a > > model. > > Often I need methods defined in models and isn't available

Re: Django ORM performance patch. Fixes #5420, #5768

2008-02-15 Thread Malcolm Tredinnick
On Fri, 2008-02-15 at 01:54 -0800, Dima Dogadaylo wrote: > On 14 Feb, 17:26, Sebastian Noack <[EMAIL PROTECTED]> > wrote: > > > > Of course, this decreases database load, but IMHO it is better to use > > QuerySet.values if you want select just certain values from a model. > > Often I need

Re: Django ORM performance patch. Fixes #5420, #5768

2008-02-15 Thread Dima Dogadaylo
On 14 Feb, 17:26, Sebastian Noack <[EMAIL PROTECTED]> wrote: > > Of course, this decreases database load, but IMHO it is better to use > QuerySet.values if you want select just certain values from a model. Often I need methods defined in models and isn't available at dictionaries. For example

Re: Django ORM performance patch. Fixes #5420, #5768

2008-02-14 Thread Sebastian Noack
> I made a patch for Django to add QuerySet.fields(*fields, > **related_fields) and make possible to load only some from master and > related models fields. It allows to tune various object list queries > when we need only limited subset of all fields, improve general > performance and decrease

Re: Django ORM performance patch. Fixes #5420, #5768

2008-02-14 Thread Malcolm Tredinnick
On Thu, 2008-02-14 at 03:50 -0800, Entropy Hacker wrote: > I made a patch for Django to add QuerySet.fields(*fields, > **related_fields) and make possible to load only some from master and > related models fields. It allows to tune various object list queries > when we need only limited subset

Django ORM performance patch. Fixes #5420, #5768

2008-02-14 Thread Entropy Hacker
I made a patch for Django to add QuerySet.fields(*fields, **related_fields) and make possible to load only some from master and related models fields. It allows to tune various object list queries when we need only limited subset of all fields, improve general performance and decrease database