From: Tomas Vondra <tomas.vondra@enterprisedb.com>
> FWIW the attached v8 patch does this - most of the conditions are moved to the
> GetModifyBatchSize() callback. I've removed the check for the BatchInsert
> callback, though - the FDW knows whether it supports that, and it seems a bit
> pointless at the moment as there are no other batch callbacks. Maybe we
> should add an Assert somewhere, though?
Thank you. I'm in favor this idea that the decision to support RETURNING and trigger is left to the FDW. I don' think
ofthe need for another Assert, as the caller has one for the returned batch size.
Regards
Takayuki Tsunakawa