From: Tomas Vondra <tomas.von...@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 the new function in 0003. God bless us. Regards Takayuki Tsunakawa
