| From: | Andres Freund <andres(at)anarazel(dot)de> |
|---|---|
| To: | Pavel Borisov <pashkin(dot)elfe(at)gmail(dot)com> |
| Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Alexander Korotkov <aekorotkov(at)gmail(dot)com>, Melanie Plageman <melanieplageman(at)gmail(dot)com>, Japin Li <japinli(at)hotmail(dot)com>, Mark Dilger <mark(dot)dilger(at)enterprisedb(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Michael Paquier <michael(at)paquier(dot)xyz>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> |
| Subject: | Re: Table AM Interface Enhancements |
| Date: | 2024-04-15 19:47:52 |
| Message-ID: | 20240415194752.lkn5v2rvqy4kjytc@awork3.anarazel.de |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
On 2024-04-15 23:14:01 +0400, Pavel Borisov wrote:
> Why it makes a difference looks a little bit unclear to me, I can't comment
> on this. I noticed that before 041b96802ef we had a block number and block
> sampler state that tied acquire_sample_rows() to the actual block
> structure.
That, and the prefetch calls actually translating the block numbers 1:1 to
physical locations within the underlying file.
And before 041b96802ef they were tied much more closely by the direct calls to
heapam added in 27bc1772fc81.
> After we have the whole struct ReadStream which doesn't comprise just a
> wrapper for the same variables, but the state that ties
> acquire_sample_rows() to the streaming read algorithm (and heap).
Yes ... ? I don't see how that is a meaningful difference to the state as of
27bc1772fc81. Nor fundamentally worse than the state 27bc1772fc81^, given
that we already issued requests for specific blocks in the file.
That said, I don't like the state after applying
https://postgr.es/m/CAPpHfdvuT6DnguzaV-M1UQ2whYGDojaNU%3D-%3DiHc0A7qo9HBEJw%40mail.gmail.com
because there's too much coupling. Hence talking about needing to iterate on
the interface in some form, earlier in the thread.
What are you actually arguing for here?
Greetings,
Andres Freund
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2024-04-15 20:02:00 | Re: Table AM Interface Enhancements |
| Previous Message | Dave Cramer | 2024-04-15 19:47:40 | Re: Add new protocol message to change GUCs for usage with future protocol-only GUCs |