Re: (select query)/relation as first class citizen

From: Dent John <denty(at)qqdd(dot)eu>
To: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
Cc: Roman Pekar <roma(dot)pekar(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: (select query)/relation as first class citizen
Date: 2019-07-10 17:01:56
Message-ID: DE237364-EB7A-4851-9337-F9F6491E46A6@qqdd.eu
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi Roman, Pavel,

I was interested in this post, as it’s a topic I’ve stumbled upon in the past.

There are two topics at play here:

1. The ability to flexibly craft queries from procedural language functions

2. Support for pipelined access to SETOF/TABLEs from procedural language functions

Postgres has parameterised functions that can return a query and optimise it in context of the overall statement, but only for the SQL language. Absence of support for other languages is a curious gap.

And it leads to fact that, even in presence of only static parameters, only one “shape” of query can ever be returned. (No IF/ELSE IF logic can alter the query, unless you dive down the rat hole of encoding it into the query itself.) So that is another gap.

Postgres has some relevant first class types: TABLE/SETOF and REFCURSOR. TABLE/SETOF are output only, materialised always and optimiser fences. Current syntax supports pipelined output (via RETURN NEXT), and docs call out the fact that it might in future not be materialised. I suspect an executor change is needed to support it, as well as plpgsql change.

Their output-only nature is an odd gap. REFCURSOR is not materialised, and is also input-capable. If SETOF/TABLE were made both, then there would be a curious type system duplication.

However REFCURSOR is pretty awkward to use from SQL. The fact you can’t cast or convert it to a SETOF/TABLE and SELECT FROM a REFCURSOR in native SQL is weird, and a gap, IMHO.

On the input aide, REFCURSOR is neat. Despite the above limitation, it can become bound to a query before being OPENed for execution and fetching. If only the optimiser could “see” that pre-OPENed state, as with parameterised views, then, in principle, there would be nothing stopping some other outer function consuming it, SELECTing FROM it, and perhaps even returning a new query, and then the optimiser would be able to see and optimise the final global statement. Okay: this is a biggie, but it’s still a gap, in my view.

So my view is that Postgres already has types that are close to what is asked for. It also has tools that look ripe to be plumbed together. Problem is, when they are combined, they don’t fit well, and when they are made to fit, the fence, materialisation always and curious output-only nature leads developers to create un-performant messes. :-)

I think some of this could be fixed quite easily. The executor already (obviously) can pipeline. PLs can’t today save and restore their context to support pipelining, but it is not impossible. REFCURSOR can’t be cast to a TABLE/SETOF, not meaningfully be SELECTed FROM, but that can’t be too hard either.

Exposing the pre-OPENed query for optimisation is another thing. But here again, I see it as a challenge of mental gymnastics rather than actually hard in terms of code factoring — much of what is needed is surely already there in the way of VIEW rewriting.

Regarding demand for the #2 feature set, this somewhat dated tread is suggestive of a niche use case: https://www.postgresql.org/message-id/flat/005701c6dc2c%2449011fc0%240a00a8c0%40trivadis.com.

d.

> On 8 Jul 2019, at 10:19, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> wrote:
>
> Hi
>
> po 8. 7. 2019 v 9:33 odesílatel Roman Pekar <roma(dot)pekar(at)gmail(dot)com> napsal:
>> Hi,
>>
>> what do you think about this idea in general? If you don't have to think about implementation for now? From my point of view writing Sql queries is very close to how functional language work if you treat "select" queries as functions without side-effects, and having query being first-class-citizen could move this even further.
>
> first - please, don't send top posts.
>
> second - my opinion is not clear. I can imagine benefits - on second hand, the usage is relative too close to one antipattern - only one query wrapped by functions. I see your proposal as little bit more dynamic (with little bit different syntax) views.
>
> With my experience I really afraid about it - it can be very effective (from developer perspective) and very slow (from customer perspective). This is example of tool that looks nice on paper, but can be very badly used.
>
> Maybe I am not the best man for this topic - I like some functional programming concepts, but I use it locally - your proposal moves SQL to some unexplored areas - and I think so it can be interesting as real research topic, but not today Postgres's theme.
>
> The basic question is why extend SQL and don't use some native functional language. Postgres should to implement ANSI SQL - and there is not a space for big experiments. I am sceptic about it - relational databases are static, SQL is static language, so it is hard to implement some dynamic system over it - SQL language is language over relation algebra - it is not functional language, I afraid so introduction another concept to this do more bad than good.
>
> Regards
>
> Pavel
>
>
>
>>
>> Regards,
>> Roman
>>
>>> On Sun, 7 Jul 2019 at 16:22, Roman Pekar <roma(dot)pekar(at)gmail(dot)com> wrote:
>>> Hi,
>>>
>>> Yes, I'm thinking about 'query like a view', 'query like a cursor' is probably possible even now in ms sql server (not sure about postgresql), but it requires this paradygm shift from set-based thinking to row-by-row thinking which I'd not want to do.
>>>
>>> I completely agree with your points of plan caching and static checks. With static checks, though it might be possible to do if the query would be defined as typed, so all the types of the columns is known in advance.
>>> In certain cases having possibility of much better decomposition is might be more important than having cached plan. Not sure how often these cases appear in general, but personally for me it'd be awesome to have this possibility.
>>>
>>> Regards,
>>> Roman Pekar
>>>
>>>> On Sun, 7 Jul 2019 at 15:39, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> wrote:
>>>> Hi
>>>>
>>>> ne 7. 7. 2019 v 14:54 odesílatel Roman Pekar <roma(dot)pekar(at)gmail(dot)com> napsal:
>>>>> Hello,
>>>>>
>>>>> Just a bit of background - I currently work as a full-time db developer, mostly with Ms Sql server but I like Postgres a lot, especially because I really program in sql all the time and type system / plpgsql language of Postgres seems to me more suitable for actual programming then t-sql.
>>>>>
>>>>> Here's the problem - current structure of the language doesn't allow to decompose the code well and split calculations and data into different modules.
>>>>>
>>>>> For example. Suppose I have a table employee and I have a function like this (I'll skip definition of return types for the sake of simplicity):
>>>>>
>>>>> create function departments_salary ()
>>>>> returns table (...)
>>>>> as
>>>>> return $$
>>>>> select department, sum(salary) as salary from employee group by department;
>>>>> $$;
>>>>>
>>>>> so that's fine, but what if I want to run this function on filtered employee? I can adjust the function of course, but it implies I can predict all possible filters I'm going to need in the future.
>>>>> And logically, function itself doesn't have to be run on employee table, anything with department and salary columns will fit.
>>>>> So it'd be nice to be able to define the function like this:
>>>>>
>>>>> create function departments_salary(_employee query)
>>>>> returns table (...)
>>>>> as
>>>>> return $$
>>>>> select department, sum(salary) as salary from _employee group by department;
>>>>> $$;
>>>>>
>>>>> and then call it like this:
>>>>>
>>>>> declare _employee query;
>>>>> ...
>>>>> _poor_employee = (select salary, department from employee where salary < 1000);
>>>>> select * from departments_salary( _poor_employee);
>>>>>
>>>>> And just to be clear, the query is not really invoked until the last line, so re-assigning _employee variable is more like building query expression.
>>>>>
>>>>> As far as I understand the closest way to do this is to put the data into temporary table and use this temporary table inside of the function. It's not exactly the same of course, cause in case of temporary tables data should be transferred to temporary table, while it will might be filtered later. So it's something like array vs generator in python, or List vs IQueryable in C#.
>>>>>
>>>>> Adding this functionality will allow much better decomposition of the program's logic.
>>>>> What do you think about the idea itself? If you think the idea is worthy, is it even possible to implement it?
>>>>
>>>>
>>>> If we talk about plpgsql, then I afraid so this idea can disallow plan caching - or significantly increase the cost of plan cache.
>>>>
>>>> There are two possibilities of implementation - a) query like cursor - unfortunately it effectively disables any optimization and it carry ORM performance to procedures. This usage is known performance antipattern, b) query like view - it should not to have a performance problems with late optimization, but I am not sure about possibility to reuse execution plans.
>>>>
>>>> Currently PLpgSQL is compromise between performance and dynamic (PLpgSQL is really static language). Your proposal increase much more dynamic behave, but performance can be much more worse.
>>>>
>>>> More - with this behave, there is not possible to do static check - so you have to find bugs only at runtime. I afraid about performance of this solution.
>>>>
>>>> Regards
>>>>
>>>> Pavel
>>>>
>>>>
>>>>>
>>>>> Regards,
>>>>> Roman Pekar
>>>>>
>>>>>

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2019-07-10 17:04:47 Re: [Proposal] Table-level Transparent Data Encryption (TDE) and Key Management Service (KMS)
Previous Message Tom Lane 2019-07-10 16:57:08 buildfarm's typedefs list has gone completely nutso