On Mon, Sep 26, 2016 at 5:03 PM, Kyotaro HORIGUCHI
<horiguchi.kyotaro@lab.ntt.co.jp> wrote:
> Hello, I return to this before my things:)
>
> Though I haven't played with the patch yet..
Be sure to run the test cases in the patch or base your tests on them then!
> Though I don't know how it actually impacts the perfomance, it
> seems to me that we can live with truncated_to and sync_above in
> RelationData and BufferNeedsWAL(rel, buf) instead of
> HeapNeedsWAL(rel, buf). Anyway up to one entry for one relation
> seems to exist at once in the hash.
TBH, I still think that the design of this patch as proposed is pretty
cool and easy to follow.
--
Michael