On 31/05/10 00:07, Tom Lane wrote: > =?UTF-8?B?SmFuIFVyYmHFhHNraQ==?= <wulc...@wulczer.org> writes: > I committed the attached revised version of the patch. Revisions are > mostly minor but I did make two substantive changes: > > * The patch changed the target number of mcelems from 10 * > statistics_target to just statistics_target. I reverted that since > I don't think it was intended; at least we hadn't discussed it.
Yeah, that was accidental. > * I modified the final processing to avoid one qsort step if there are > fewer than num_mcelems hashtable entries that pass the cutoff frequency > filter, and in any case to sort only those entries that pass it rather > than all of them. With the significantly larger number of hashtable > entries that will now be used, it seemed like a good thing to try to > cut the qsort overhead. Make sense. Thanks, Jan -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
