Re: Pluggable Storage - Andres's take

On Thu, Oct 18, 2018 at 6:28 AM Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com> wrote:
> On Tue, Oct 16, 2018 at 6:06 AM Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru> wrote:
>> * 0002-add-costing-function-to-API.patch – adds function for costing
>> sequential and table sample scan to tableam API. zheap costing
>> function are now copies of heap costing function. This should be
>> adjusted in future.
>
> This patch misses the new *_cost.c files that are added specific cost
> functions.

Thank you for noticing. Revised patchset is attached.

>> Estimation for heap lookup during index scans
>> should be also pluggable, but not yet implemented (TODO).
>
> Yes, Is it possible to use the same API that is added by above
> patch?

I'm not yet sure. I'll elaborate more on that. I'd like to keep
number of costing functions small. Handling of costing of index scan
heap fetches will probably require function signature change.

------
Alexander Korotkov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company