From: | "Timmer, Marius" <marius(dot)timmer(at)uni-muenster(dot)de> |
---|---|
To: | Arne Scheffer <scheffa(at)uni-muenster(dot)de> |
Cc: | Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>, "Timmer, Marius" <marius(dot)timmer(at)uni-muenster(dot)de>, pgsql-hackerspostgresqlorg <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [PATCH] explain sortorder |
Date: | 2015-01-15 15:05:54 |
Message-ID: | 68A3685D-858C-4EE0-BC59-73447EC49C43@exchange.wwu.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
attached is version 8, fixing remaining issues, adding docs and tests as requested/agreed.
Marius & Arne
---
Marius Timmer
Zentrum für Informationsverarbeitung
Westfälische Wilhelms-Universität Münster
Einsteinstraße 60
mtimm_01(at)uni-muenster(dot)de
Am 14.01.2015 um 17:42 schrieb Arne Scheffer <scheffa(at)uni-muenster(dot)de>:
> Hi,
>
> we will also remove the following is lc_collate hint in the next version, showing only mandatory info as suggested.
>
> /* for those who use COLLATE although their default is already the wanted */
> if (strcmp(collname, localeptr) == 0)
> {
> appendStringInfo(sortorderInformation, " (%s is LC_COLLATE)", collname);
> }
>
> Anybody insisting on that?
>
> Arne
>
> Note: I see, at the moment we use the wrong default for DESC. We'll fix that.
>
> On Wed, 14 Jan 2015, Heikki Linnakangas wrote:
>
>> On 01/14/2015 05:26 PM, Timmer, Marius wrote:
>>> Hello Heikki,
>>> abbreviated version:
>>> Sorry, the problem is only the unhandy patch text format, not different opinions how to proceed.
>>> Long version:
>>> The v7 patch file already addressed your suggestions,
>>> but the file contained serveral (old) local commits,
>>> the new ones at the end of the patch text/file.
>>
>> Ah, missed that. I stopped reading when I saw the old stuff there :-).
>>
>>> v7.1 is attached and addresses this issue providing a clean patch file.
>>
>> Ok, thanks, will take a look.
>>
>>> V8 will - as mentioned - add missing docs and regression tests,
>>
>> Great!
>>
>> - Heikki
>>
>>
Attachment | Content-Type | Size |
---|---|---|
explain_sortorder-v8.patch | application/octet-stream | 14.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2015-01-15 15:53:42 | Re: TODO : Allow parallel cores to be used by vacuumdb [ WIP ] |
Previous Message | Robert Haas | 2015-01-15 14:55:08 | Re: Safe memory allocation functions |