> The patch as given has a bunch of implementation issues, but I think > it's close enough for crude performance testing, and your numbers do > show a potential performance benefit. The question that I think is > unresolved is whether the set of cases covered is wide enough to be > useful in practice. I have no data on that ...
Well, I think its use for timestamp/interval/casting functions alone covers a pretty substantial set of common user actions. Frankly, the case of not re-evaluating now() alone would be a significant real-life improvement. If I understand the limitations correctly, though, what this would do is cause functions to perform substantially differently if called with expressions as arguments instead of text constants, no? Seems like that would lead to some user confusion. Although, with stuff like now(), we already have that. -- Josh Berkus PostgreSQL Experts Inc. http://pgexperts.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers