| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
| Cc: | Amit Langote <amitlangote09(at)gmail(dot)com>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, "houzj(dot)fnst(at)fujitsu(dot)com" <houzj(dot)fnst(at)fujitsu(dot)com>, "tanghy(dot)fnst(at)fujitsu(dot)com" <tanghy(dot)fnst(at)fujitsu(dot)com>, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org>, "shiy(dot)fnst(at)fujitsu(dot)com" <shiy(dot)fnst(at)fujitsu(dot)com> |
| Subject: | Re: Table refer leak in logical replication |
| Date: | 2021-04-19 06:12:36 |
| Message-ID: | YH0fVFC7uE049L5C@paquier.xyz |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, Apr 19, 2021 at 03:08:41PM +0900, Michael Paquier wrote:
> FWIW, I
> would be tempted to send back f1ac27b to the blackboard, then refactor
> the code of the apply worker to use ExecInitResultRelation() so as we
> get more consistency with resource releases, simplifying the business
> with indexes. Once the code is in a cleaner state, we could come back
> into making an integration with partitioned tables into this code.
But you cannot do that either as f1ac27bf got into 13..
--
Michael
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Amul Sul | 2021-04-19 06:24:00 | Re: Remove redundant variable from transformCreateStmt |
| Previous Message | Michael Paquier | 2021-04-19 06:08:41 | Re: Table refer leak in logical replication |