From: Tomas Vondra <tomas.von...@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 of the need for another Assert, as the caller has one for the returned batch size. Regards Takayuki Tsunakawa
