On 25.01.2011 06:29, Pavel Stehule wrote:
2011/1/25 Noah Misch<n...@leadboat.com>:
On Sat, Jan 22, 2011 at 11:32:02AM +0100, Pavel Stehule wrote:
because I am not sure so any complex solution can be done to deadline
for 9.1, I created a patch that is based on Tom ideas - just
explicitly detoast function parameters.

I can confirm that, for your original test case, this yields performance
comparable to that of your original patch.

I know it :(. I am thinking, so detoasting on usage is better, but I
am don't know more about Tom or Rober's plans.

Detoasting on first usage, ie. exec_eval_datum(), seems the best to me. Compared to detoasting on assignment, it avoids the performance regression if the value is never used, and I don't think checking if the value is toasted at every exec_eval_datum() call adds too much overhead.

--
  Heikki Linnakangas
  EnterpriseDB   http://www.enterprisedb.com

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to