From: Tomas Vondra <tomas.vondra@enterprisedb.com>
> I took a look at this - there's a bit of bitrot due to 708d165ddb92c, so attached is
> a rebased patch (0001) fixing that.
>
> 0002 adds a couple comments and minor tweaks
>
> 0003 addresses a couple shortcomings related to explain - we haven't been
> showing the batch size for EXPLAIN (VERBOSE), because there'd be no
> FdwState, so this tries to fix that. Furthermore, there were no tests for EXPLAIN
> output with batch size, so I added a couple.
Thank you, good additions. They all look good.
Only one point: I think the code for retrieving batch_size in create_foreign_modify() can be replaced with a call to
thenew function in 0003.
God bless us.
Regards
Takayuki Tsunakawa