From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Heikki Linnakangas <heikki(at)enterprisedb(dot)com> |
Cc: | pgsql-patches(at)postgreSQL(dot)org |
Subject: | Re: WIP: 2nd-generation buffer ring patch |
Date: | 2007-05-30 18:39:46 |
Message-ID: | 9998.1180550386@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Heikki Linnakangas <heikki(at)enterprisedb(dot)com> writes:
> Tom Lane wrote:
>> The COPY IN part of the patch is not there, pending resolution of
>> whether we think it adds enough value to be worth uglifying
>> heap_insert's API for.
> I ran a series of tests, and it looks like it's not worth it.
Great, I'll pull out the last vestiges of that and apply.
If we did want to pursue this, I was thinking of inventing a
"BulkInsertTarget" object type that could be passed to heap_insert,
in the same spirit as BufferAccessStrategy in my WIP patch.
This would carry a BufferAccessStrategy and also serve to track a
currently-pinned target page as per your thought about avoiding
pin/unpin cycles across multiple inserts. I think we could fold
use_fsm into it as well (maybe use_wal too), and thereby avoid growing
heap_insert's parameter list still more.
Not something I want to pursue now, but just getting these thoughts
into the archives in case someone picks it up again.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Neil Conway | 2007-05-30 19:02:04 | Re: boolean <=> text explicit casts |
Previous Message | Heikki Linnakangas | 2007-05-30 18:27:46 | Re: Seq scans status update |