[ in the service of closing out this thread... ] Peter Geoghegan <p...@heroku.com> writes: > Finally, 0003-* is a Valgrind suppression borrowed from my parallel > CREATE INDEX patch. It's self-explanatory.
Um, I didn't find it all that self-explanatory. Why wouldn't we want to avoid writing undefined data? I think the comment at least needs to explain exactly what part of the written data might be uninitialized. And I'd put the comment into valgrind.supp, too, not in the commit msg. Also, the suppression seems far too broad. It would for instance block any complaint about a write() invoked via an elog call from any function invoked from any LogicalTape* function, no matter how far removed. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers