Re: Optimze usage of immutable functions as relation - Mailing list pgsql-hackers

From Antonin Houska
Subject Re: Optimze usage of immutable functions as relation
Date
Msg-id 11120.1541686083@localhost
Whole thread Raw
In response to Re: Optimze usage of immutable functions as relation  (Aleksandr Parfenov <asp437@gmail.com>)
Responses Re: Optimze usage of immutable functions as relation
List pgsql-hackers
Aleksandr Parfenov <asp437@gmail.com> wrote:

> I fixed a typo and some comments. Please find new version attached.

I've checked this verstion too.

* is_simple_stable_function()

instead of fetching HeapTuple from the syscache manually, you might want to
consider using functions from lsyscache.c (get_func_rettype, get_func_retset,
etc.), or adding a function that returns (subset of) the fields you need in a
single call.

* pull_up_simple_function():

As you assume that ret->functions is a single-item list

    Assert(list_length(rte->functions) == 1);

the following iteration is not necessary:

    foreach(lc, functions_list)

Also, there seems to be a lot of copy & paste from pull_up_simple_values(), so
some refactoring would make sense.


--
Antonin Houska
Cybertec Schönig & Schönig GmbH
Gröhrmühlgasse 26, A-2700 Wiener Neustadt
Web: https://www.cybertec-postgresql.com


pgsql-hackers by date:

Previous
From: Antonin Houska
Date:
Subject: Re: Optimze usage of immutable functions as relation
Next
From: Markus Winand
Date:
Subject: Re: PostgreSQL vs SQL/XML Standards