Heikki Linnakangas <[EMAIL PROTECTED]> writes:
> The FSM would be perfectly happy to just initialize torn or otherwise 
> damaged pages, so I think we should add yet another mode to ReadBuffer() 
> to allow that. We could also treat read() errors as merely warnings in 
> that mode, effectively the same as with zero_damaged_pages=on.

> The ReadBuffer() interface is already pretty complex, with all the 
> different variants. We should probably keep the good old ReadBuffer() 
> the same, for the sake of simplicity in the callers, but try to reduce 
> the number of other variatns.

Indeed.  Did you see the discussion about the similarly-too-complex
heap_insert API a couple days ago in connection with bulk-write
scenarios?  The conclusion there was to try to shift stuff into a
bitmask options argument, in hopes that future additions might not
require touching every caller.  Can we do it similarly here?

                        regards, tom lane

-- 
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