| From: | Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com> |
|---|---|
| To: | Julien Rouhaud <rjuju123(at)gmail(dot)com> |
| Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Daniel Verite <daniel(at)manitou-mail(dot)org> |
| Subject: | Re: ICU for global collation |
| Date: | 2022-01-27 08:10:46 |
| Message-ID: | 87e93704-b7f7-71f6-161d-8713e6334ed3@enterprisedb.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 21.01.22 17:13, Julien Rouhaud wrote:
> On Fri, Jan 21, 2022 at 03:24:02PM +0100, Peter Eisentraut wrote:
>> On 21.01.22 14:51, Julien Rouhaud wrote:
>>> Is that change intended? There isn't any usage of the collversionstr before
>>> the possible error when actual_versionstr is missing.
>>
>> I wanted to move it closer to the SysCacheGetAttr() where the "datum" value
>> is obtained. It seemed weird to get the datum, then do other things, then
>> decode the datum.
>
> Oh ok. It won't make much difference performance-wise, so no objection.
I have committed this and will provide follow-up patches in the next few
days.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Anand Sowmithiran | 2022-01-27 08:21:10 | Re: Output clause for Upsert aka INSERT...ON CONFLICT |
| Previous Message | Kyotaro Horiguchi | 2022-01-27 08:10:42 | Re: Support escape sequence for cluster_name in postgres_fdw.application_name |