Re: Trivial function query optimized badly - Mailing list pgsql-performance

From Craig A. James
Subject Re: Trivial function query optimized badly
Date
Msg-id 459CA541.3060009@modgraph-usa.com
Whole thread Raw
In response to Re: Trivial function query optimized badly  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-performance
Tom Lane wrote:
> "Craig A. James" <cjames@modgraph-usa.com> writes:
>>   CREATE OR REPLACE FUNCTION cansmiles(text) RETURNS text
>>   AS '/usr/local/pgsql/lib/libchem.so', 'cansmiles'
>>   LANGUAGE 'C' STRICT IMMUTABLE;
>
> Umm ... this is a single-argument function.
>
>> db=> explain analyze select version_id, 'Brc1ccc2nc(cn2c1)C(=O)O' from version where version.isosmiles =
cansmiles('Brc1ccc2nc(cn2c1)C(=O)O',1); 
>
> And this query is invoking some other, two-argument function; which
> apparently hasn't been marked IMMUTABLE, else it'd have been folded
> to a constant.

Good catch, mystery solved.  There are two definitions for this function, the first just a "wrapper" for the second
withthe latter parameter defaulting to "1".  The second definition was missing the "IMMUTABLE" keyword. 

Thanks!
Craig

pgsql-performance by date:

Previous
From: Tom Lane
Date:
Subject: Re: Trivial function query optimized badly
Next
From: "Jeremy Haile"
Date:
Subject: Re: Performance of PostgreSQL on Windows vs Linux