From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | "Heikki Linnakangas" <heikki(at)enterprisedb(dot)com> |
Cc: | "Simon Riggs" <simon(at)2ndquadrant(dot)com>, "Hell, Robert" <Robert(dot)Hell(at)fabasoft(dot)com>, pgsql-patches(at)postgresql(dot)org, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [HACKERS] GUC parameter cursors_tuple_fraction |
Date: | 2008-05-02 16:01:37 |
Message-ID: | 6871.1209744097@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
"Heikki Linnakangas" <heikki(at)enterprisedb(dot)com> writes:
> Simon Riggs wrote:
>> * We've said here http://www.postgresql.org/docs/faqs.TODO.html that we
>> "Don't want hints". If that's what we really think, then this patch must
>> surely be rejected because its a hint... That isn't my view. I *now*
>> think we do need hints of various kinds.
> cursors_tuple_fraction or OPTIMIZE FOR xxx ROWS isn't the kind of hints
> we've said "no" to in the past.
More to the point, I think what we've generally meant by "hints" is
nonstandard decoration on individual SQL commands (either explicit
syntax or one of those interpret-some-comments kluges). Simon is
reading the policy in such a way that it would forbid all the planner
cost parameters, which is surely not what is intended.
I see this as being basically another cost parameter, and as such
I don't think it needs more documentation than any of those have.
(Now admittedly you could argue that they could all use a ton more
documentation than they now have, but it's not reasonable to insist
on just this one meeting a different standard.)
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Greg Sabino Mullane | 2008-05-02 17:26:14 | Re: Protection from SQL injection |
Previous Message | Heikki Linnakangas | 2008-05-02 15:17:14 | Re: GUC parameter cursors_tuple_fraction |
From | Date | Subject | |
---|---|---|---|
Next Message | Mark Wong | 2008-05-02 16:12:32 | Re: configure option for XLOG_BLCKSZ |
Previous Message | Tom Lane | 2008-05-02 15:50:42 | Re: configure option for XLOG_BLCKSZ |