Postgres Pro
Downloads
Home   >   mailing lists

Re: Custom table AMs need to include heapam.h because of BulkInsertState - Mailing list pgsql-hackers

From David Rowley
Subject Re: Custom table AMs need to include heapam.h because of BulkInsertState
Date
Msg-id CAKJS1f_ZQLCEnmSRZwnbs4xw8VDvh7L2sOoA6oNzhUOQ_ZFm0Q@mail.gmail.com
Whole thread Raw
In response to Re: Custom table AMs need to include heapam.h because ofBulkInsertState  (Andres Freund <andres@anarazel.de>)
List pgsql-hackers
Tree view 
On Wed, 17 Jul 2019 at 06:46, Andres Freund <andres@anarazel.de> wrote:
> 2) Add a execPartition.c function that returns all the used tables from
>    a PartitionTupleRouting*.

Here's a patch which implements it that way.

I struggled a bit to think of a good name for the execPartition.c
function. I ended up with ExecGetRoutedToRelations. I'm open to better
ideas.

I also chose to leave the change of function signatures done in
f7c830f1a in place. I don't think the additional now unused parameter
is that out of place. Also, the function is inlined, so removing it
wouldn't help performance any.

-- 
 David Rowley                   http://www.2ndQuadrant.com/
 PostgreSQL Development, 24x7 Support, Training & Services
Attachment

pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: use of the term "verifier" with SCRAM
Next
From: Alex
Date:
Subject: use valgrind --leak-check=yes to detect memory leak