On 1/7/15, 3:50 AM, Marco Nenciarini wrote:
The current implementation tracks only heap LSN. It currently does not
track any kind of indexes, but this can be easily added later.

Would it make sense to do this at a buffer level, instead of at the heap level? 
That means it would handle both heap and indexes. I don't know if LSN is 
visible that far down though.

Also, this pattern is repeated several times; it would be good to put it in 
it's own function:
+               lsnmap_pin(reln, blkno, &lmbuffer);
+               lsnmap_set(reln, blkno, lmbuffer, lsn);
+               ReleaseBuffer(lmbuffer);
--
Jim Nasby, Data Architect, Blue Treble Consulting
Data in Trouble? Get it in Treble! http://BlueTreble.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