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