On Fri, 2007-03-02 at 15:49 -0800, Josh Berkus wrote:
> Jeff,
> 
> > Right now they are just constants defined in a header, but a GUC might
> > make sense. I'd like to know which version is more acceptable when I
> > submit my final patch.
> 
> As much as I hate the thought of more GUCs, until we have a solid 
> performance profile for synch scan we probably need them.  You should 

I will include them in the final patch then. 

> include the option to turn synch_scan off, such as by setting 
> synch_scan_threshold to -1.

Naturally.

> Oh, and remember that these now need to be able to take K/MB/GB.

Will do.

> These options should probably go in postgresql.conf under QUERY TUNING, 
> with their own sub-head.

That makes sense to me.

Regards,
        Jeff Davis

PS: Did you happen to get my patch for testing (sent off-list)? If
testing will take a while, that's OK, I'd just like to know whether to
expect the results before feature freeze.


---------------------------(end of broadcast)---------------------------
TIP 6: explain analyze is your friend

Reply via email to