On Fri, Feb 2, 2018 at 1:58 PM, Andres Freund <and...@anarazel.de> wrote: > Not saying you're wrong, but you should include a comment on why this is > a benign warning. Presumably it's some padding memory somewhere, but > it's not obvious from the above bleat.
Sure. This looks slightly more complicated than first anticipated, but I'll keep everyone posted. Valgrind suppression aside, this raises another question. The stack trace shows that the error happens during the creation of a new TOAST table (CheckAndCreateToastTable()). I wonder if I should also pass down a flag that makes sure that parallelism is never even attempted from that path, to match TRUNCATE's suppression of parallel index builds during its reindexing. It really shouldn't be a problem as things stand, but maybe it's better to be consistent about "useless" parallel CREATE INDEX attempts, and suppress them here too. -- Peter Geoghegan
