Re: POC: postgres_fdw insert batching
Zhihong Yu <zyu@yugabyte.com>
From: Zhihong Yu <zyu@yugabyte.com>
To: "tsunakawa.takay@fujitsu.com" <tsunakawa.takay@fujitsu.com>
Cc: Tomas Vondra <tomas.vondra@enterprisedb.com>,
Amit Langote <amitlangote09@gmail.com>, "Andrey V. Lepikhov" <a.lepikhov@postgrespro.ru>,
Tomas Vondra <tomas.vondra@2ndquadrant.com>, "pgsql-hackers@postgresql.org" <pgsql-hackers@postgresql.org>
Date: 2021-01-21T01:49:00Z
Lists: pgsql-hackers
Hi, Takayuki-san: My first name is Zhihong. You can call me Ted if you want to save some typing :-) Cheers On Wed, Jan 20, 2021 at 5:37 PM tsunakawa.takay@fujitsu.com < tsunakawa.takay@fujitsu.com> wrote: > From: Zhihong Yu <zyu@yugabyte.com> > > > Do we need to consider how this part of code inside > ExecInitModifyTable() would evolve ? > > > > > I think placing the compound condition toward the end of > ExecInitModifyTable() is reasonable because it checks the latest > information. > > > > +1 for Zaihong-san's idea. But instead of rewriting every relsultRelInfo > to mtstate->resultRelInfo, which makes it a bit harder to read, I'd like to > suggest just adding "resultRelInfo = mtstate->resultRelInfo;" immediately > before the if block. > > > > Thanks a lot, all for helping to solve the problem quickly! > > > > > > Regards > > Takayuki Tsunakawa > > > >
Commits
-
Fix tuple routing to initialize batching only for inserts
- 927f453a9410 14.0 landed
-
Fix pointer type in ExecForeignBatchInsert SGML docs
- c15283ff429b 14.0 landed
-
Fix initialization of FDW batching in ExecInitModifyTable
- 920f853dc948 14.0 landed
-
Implement support for bulk inserts in postgres_fdw
- b663a4136331 14.0 landed