On Fri, Feb 20, 2015 at 4:11 PM, Tomas Vondra <tomas.von...@2ndquadrant.com> wrote: >> So you're testing both the patches (numeric + datum tuplesort) at the >> same time? > > No, I was just testing two similar patches separately. I.e. master vs. > each patch separately.
Well, you're sorting numeric here, no? Why should it matter that a datum sort has abbreviation support, if the underlying type (numeric) does not support abbreviation? OTOH, why should having oplcass abbreviation support (for numeric) matter if the class of tuple sorted (datum "tuples") does not support abbreviation? You need both to meaningfully benchmark either (as long as you're looking at a case involving both). I suggest looking at datum sorts with text for the datum sort patch, and non-datum tuplesort cases for the numeric patch, at least until such time as one or the other is committed. -- 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
