On Wed, Sep 6, 2017 at 7:26 PM, Peter Geoghegan <p...@bowt.ie> wrote: > On Wed, Aug 30, 2017 at 9:29 AM, Peter Geoghegan <p...@bowt.ie> wrote: >> On Wed, Aug 30, 2017 at 5:02 AM, Alvaro Herrera >> <alvhe...@2ndquadrant.com> wrote: >>> Eh, if you want to optimize it for the case where debug output is not >>> enabled, make sure to use ereport() not elog(). ereport() >>> short-circuits evaluation of arguments, whereas elog() does not. >> >> I should do that, but it's still not really noticeable. > > Since this patch has now bit-rotted, I attach a new revision, V2.
I should point out that I am relying on deterministic TOAST compression within index_form_tuple() at present. This could, in theory, become a problem later down the road, when toast_compress_datum() compression becomes configurable via a storage parameter or something (e.g., we use PGLZ_strategy_always, rather than the hard coded PGLZ_strategy_default strategy). While I should definitely have a comment above the new amcheck index_form_tuple() call that points this out, it's not clear if that's all that is required. Normalizing the representation of hashed index tuples to make verification robust against unforeseen variation in TOAST compression strategy seems like needless complexity to me, but I'd like to hear a second opinion on that. -- Peter Geoghegan -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
