> On 25 Feb 2020, at 00:29, Andres Freund <and...@anarazel.de> wrote: > On 2020-02-23 16:27:57 +0900, Michael Paquier wrote:
>> Good catch. I would not backpatch that as it is not a live bug >> because heap_multi_insert() is not used for catalogs yet. With your >> patch, that would be the case though.. > > Thanks for pushing. +1, thanks to the both of you for helping with the patch. Attached is a v8 of the patchset to make the cfbot happier, as the previous no longer applies. In doing that I realized that there is another hunk in this patch for fixing up logical decoding multi-insert support, which is independent of the patch in question here so I split it off. It's another issue which cause no harm at all today, but fails as soon as someone tries to perform multi inserts into the catalog. AFAICT, the current coding assumes that the multi-insert isn't from a catalog, and makes little attempts at surviving in case it is. The attached patch fixes that by taking a fast-path exit in case it is a catalog multi-insert, as there is nothing to decode. cheers ./daniel
catalog_multi_insert-v8.patch
Description: Binary data
decodemultiinsert_tupledata.patch
Description: Binary data
