From: Heikki Linnakangas <hlinnakangas@vmware.com>
To: Matheus de Oliveira <matioli.matheus@gmail.com>
Cc: pgsql-performance <pgsql-performance@postgresql.org>
Cc: Charles Gomes <charlesrg@outlook.com>
Subject: Re: Partition insert trigger using C language
Date: Fri, 11 Jan 2013 13:02:29 +0200
Message-ID: <50EFF145.1050306@vmware.com> (raw)
In-Reply-To: <CAJghg4+=Gx5AxHmEx=f6wLGP-WL-wmaLrRZqBopgoo6YbxmCyQ@mail.gmail.com>
References: <CAJghg4KJToKVdr=uecKyOYUjx5h3iNdxYPyhLst=8Lv78Oac+w@mail.gmail.com>
<50EF0E81.1010808@vmware.com>
<CAJghg4LeodBzGa=9D11HK+v2+ttiFFQtxKwEBKMo-3RDPNQiyg@mail.gmail.com>
<50EF14F1.5070901@vmware.com>
<CAJghg4LwS_2+987yDvDW3+Hc2u-t1CW-9uOrd-QS870piJ8S_A@mail.gmail.com>
<50EFE74B.9040007@vmware.com>
<CAJghg4+=Gx5AxHmEx=f6wLGP-WL-wmaLrRZqBopgoo6YbxmCyQ@mail.gmail.com>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-performance>
On 11.01.2013 12:36, Matheus de Oliveira wrote:
> On Fri, Jan 11, 2013 at 8:19 AM, Heikki Linnakangas<hlinnakangas@vmware.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
--
Sent via pgsql-performance mailing list (pgsql-performance@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-performance
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-performance@postgresql.org
Cc: hlinnakangas@vmware.com, matioli.matheus@gmail.com, charlesrg@outlook.com
Subject: Re: Partition insert trigger using C language
In-Reply-To: <50EFF145.1050306@vmware.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox