On Wed, May 13, 2015 at 2:44 PM, Peter Geoghegan wrote:
> On Wed, May 13, 2015 at 11:40 AM, Robert Haas wrote:
>> Committed. Thanks for the patch and your patience.
>
> This comment was not updated:
>
> /*
> * The sortKeys variable is used by every case other than the datum and
> * hash index
On Wed, May 13, 2015 at 11:40 AM, Robert Haas wrote:
> Committed. Thanks for the patch and your patience.
This comment was not updated:
/*
* The sortKeys variable is used by every case other than the datum and
* hash index cases; it is set by tuplesort_begin_xxx. tupDesc is only
* used by t
On Fri, Jan 23, 2015 at 4:13 PM, Andrew Gierth
wrote:
> [pruning the Cc: list and starting a new thread]
>
> Here's the cleaned-up version of the patch to allow abbreviated keys
> when sorting a single Datum. This also removes comments that suggest
> that the caller of tuplesort_begin_datum should
On Sat, Jan 24, 2015 at 6:19 PM, Robert Haas wrote:
> I think this is a good idea. Do you have a test case that shows the benefit?
I agree. It seems likely that this will show a similar benefit to
other cases already tested, but I'd like to see a test case too.
--
Peter Geoghegan
--
Sent vi
On Fri, Jan 23, 2015 at 4:13 PM, Andrew Gierth
wrote:
> [pruning the Cc: list and starting a new thread]
>
> Here's the cleaned-up version of the patch to allow abbreviated keys
> when sorting a single Datum. This also removes comments that suggest
> that the caller of tuplesort_begin_datum should
