Remove unnecessary list_free() calls in OpenTableList(). OpenTableList() is invoked at most once per DDL statement, from CreatePublication() and AlterPublicationTables(), and everything it builds (relids, relids_with_rf, and relids_with_collist) is allocated in the command's own memory context. That context is reclaimed as soon as the command finishes, so explicitly freeing relids and relids_with_rf here served no purpose.
Reported-by: Peter Smith <[email protected]> Suggested-by: Tom Lane <[email protected]> Author: Peter Smith <[email protected]> Reviewed-by: shveta malik <[email protected]> Reviewed-by: shihao zhong <[email protected]> Reviewed-by: surya poondla <[email protected]> Discussion: https://postgr.es/m/CAHut+Pt3bgyt==d6GCj=p9qu4svsh_kqwln7cmah0rdjco-...@mail.gmail.com Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/cc1ea710dc88186aab4b3c129883d710781fcfb6 Modified Files -------------- src/backend/commands/publicationcmds.c | 3 --- 1 file changed, 3 deletions(-)
