On Thu, Aug 22, 2024 at 7:31 PM Mats Kindahl <m...@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.
