Etsuro Fujita wrote:
> Please find attached the updated version of the patch.
I gave it a spin and could not find any undesirable behaviour, and the
output of EXPLAIN ANALYZE looks like I'd expect.
I noticed that you use the list length of fdw_private to check if
the UPDATE or DELETE is pushed down to the remote server or not.
While this works fine, I wonder if it wouldn't be better to have some
explicit flag in fdw_private for that purpose. Future modifications that
change the list length might easily overlook that it is used for this
purpose, thereby breaking the code.
Other than that it looks alright to me.
Yours,
Laurenz Albe
