Re: [HACKERS] Pluggable storage

On Wed, Dec 27, 2017 at 11:33 PM, Alexander Korotkov <a.korotkov@postgrespro.ru> wrote:

Also, I appreciate that now tuple_insert() and tuple_update() methods are responsible for inserting index tuples.  This unleash pluggable storages to implement another way of interaction with indexes.  However, I didn't get the point of passing InsertIndexTuples IndexFunc to them.  Now, we're always passing ExecInsertIndexTuples() to this argument.  As I understood storage is free to either call ExecInsertIndexTuples() or implement its own logic of interaction with indexes.  But, I don't understand why do we need a callback when tuple_insert() and tuple_update() can call ExecInsertIndexTuples() directly if needed.  Another thing is that tuple_delete() could also interact with indexes (especially when we will enhance index access method API), and we need to pass meta-information about indexes to tuple_delete() too.

The main reason for which I added the callback function to not to introduce the

dependency of storage on executor functions. This way storage can call the

function that is passed to it without any knowledge. I added the function pointer

for tuple_delete also in the new patches, currently it is passed as NULL for heap.

These API's can be enhanced later.

Apart from rebase, Added storage shared memory API, currently this API is used

only by the syncscan. And also all the exposed functions of syncscan usage is

removed outside the heap.