From: | Heikki Linnakangas <hlinnakangas(at)vmware(dot)com> |
---|---|
To: | Matheus de Oliveira <matioli(dot)matheus(at)gmail(dot)com> |
Cc: | pgsql-performance <pgsql-performance(at)postgresql(dot)org>, Charles Gomes <charlesrg(at)outlook(dot)com> |
Subject: | Re: Partition insert trigger using C language |
Date: | 2013-01-11 11:02:29 |
Message-ID: | 50EFF145.1050306@vmware.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-performance |
On 11.01.2013 12:36, Matheus de Oliveira wrote:
> On Fri, Jan 11, 2013 at 8:19 AM, Heikki Linnakangas<hlinnakangas(at)vmware(dot)com
>> wrote:
>
>> One thing that caught my eye:
>>
>> CREATE OR REPLACE FUNCTION partition_insert_trigger_spi()
>>> RETURNS trigger
>>> LANGUAGE C
>>> VOLATILE STRICT
>>> AS 'partition_insert_trigger_spi'**,'partition_insert_trigger_**spi'
>>> SET DateStyle TO 'ISO';
>>
>> Calling a function with SET options has a fair amount of overhead, to
>> set/restore the GUC on every invocation. That should be avoided in a
>> performance critical function like this.
>
> I (stupidly) used SPI_getvalue [1] and expected it to always return as
> YYYY-MM-DD, but them I remembered it would do that only with DateStyle=ISO.
>
> But the truth is that I couldn't see any overhead, because the function was
> without that on my first tests, and after that I saw no difference on the
> tests.
Oh, ok then. I would've expected it to make a measurable difference.
> I think I should use SPI_getbinvalue instead, but I don't know how
> to parse the result to get year and month, any help on that?
The fastest way is probably to use j2date like date_out does:
DateADT date = DatumGetDateADT(x)
int year, month, mday;
if (DATE_NOT_FINITE(date))
elog(ERROR, "date must be finite");
j2date(date + POSTGRES_EPOCH_JDATE, &year, &month, &mday);
- Heikki
From | Date | Subject | |
---|---|---|---|
Next Message | Matheus de Oliveira | 2013-01-11 11:55:07 | Re: Partition insert trigger using C language |
Previous Message | Matheus de Oliveira | 2013-01-11 10:57:15 | Re: Partition insert trigger using C language |