On Thu, Aug 22, 2024 at 7:31 PM Mats Kindahl <mats@timescale.com> wrote:
> The alternate version proposed by Nazir allows you to decide which interface to use.
> Reverting the patch entirely would also solve the problem.
> Passing down the block sampler and the strategy to scan_begin() and move the ReadStream setup in analyze.c into
initscan()in heapam.c, but this requires adding new parameters to this function.
> Having accessors that allow you to get the block sampler and strategy from the ReadStream object.
I'm a bit confused about how it can make sense to use the same
BlockSampler with a side relation/fork. Could you point me at the
code?
> It would be great if this could be fixed before the PG17 release now that 27bc1772fc8 was reverted.
Ack. Thinking...
Random thought: is there a wiki page or something where we can find
out about all the table AM projects? For the successor to
27bc1772fc8, I hope they'll be following along.