Peter Eisentraut <[EMAIL PROTECTED]> writes:
> Pavel Stehule wrote:
>> I did task from TODO: Add BETWEEN ASYMMETRIC/SYMMETRIC.
> The implementation in this patch has the same problems as all the
> previously rejected attempts: it evaluates its arguments twice. You
> need to make BETWEEN SYMMETRIC into a separate node type that evaluates
> each argument only once.
Sure, but BETWEEN already does that; Pavel's not making that situation
any worse. The trouble with making it a separate node type is that that
raises the stakes a great deal concerning the amount of planner support
needed (assuming you still want indexability...)
regards, tom lane
---------------------------(end of broadcast)---------------------------
TIP 3: if posting/reading through Usenet, please send an appropriate
subscribe-nomail command to [EMAIL PROTECTED] so that your
message can get through to the mailing list cleanly