pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Gilles Darold <gilles@darold.net>
To: pgsql-hackers@lists.postgresql.org
Subject: Re: Deparsing rewritten query
Date: Mon, 28 Jun 2021 16:06:54 +0200
Message-ID: <8f0b88a8-e9b8-0fd8-4dd1-ad14c4fdca0e@darold.net> (raw)
In-Reply-To: <8085.1624806845@sss.pgh.pa.us>
References: <20210627041138.zklczwmu3ms4ufnk@nol>
	<3176554.1624804492@sss.pgh.pa.us>
	<20210627150343.zslwnw7g564f2zz4@nol>
	<8085.1624806845@sss.pgh.pa.us>

Le 27/06/2021 à 17:14, Tom Lane a écrit :
> Julien Rouhaud <rjuju123@gmail.com> writes:
>> On Sun, Jun 27, 2021 at 10:34:52AM -0400, Tom Lane wrote:
>>> It's not very hard to imagine someday moving view
>>> expansion into the planner on efficiency grounds, leaving the rewriter
>>> handling only the rare uses of INSERT/UPDATE/DELETE rules.
>> Agreed.  One the other hand having such a function in core may ensure that any
>> significant change in those area will keep an API to retrieve the final query
>> representation.
> My point is precisely that I'm unwilling to make such a promise.
>
> I do not buy that this capability is worth very much, given that
> we've gotten along fine without it for twenty-plus years.  If you
> want to have it as an internal, might-change-at-any-time API,
> that seems all right.  If you're trying to lock it down as something
> that will be there forevermore, you're likely to end up with nothing.
>
> 			regards, tom lane


I have to say that such feature would be very helpful for some DBA and 
especially migration work. The problem is when you have tons of views 
that call other views in the from or join clauses. These views also call 
other views, etc. I have had instances where there were up to 25 nested 
views calls. When you want to clean up this kind of code, using 
get_query_def () will help save a lot of manual rewrite time and 
headache to get the final code executed.


If we could at least call get_query_def()through an extension if we 
didn't have a functionit would be ideal for DBAs.I agree this is unusual 
but when it does happen to you being able to call get_query_def () helps 
a lot.


-- 
Gilles Darold
http://www.darold.net/

view thread (24+ messages)  latest in thread

Message-ID: <8f0b88a8-e9b8-0fd8-4dd1-ad14c4fdca0e@darold.net>
Permalink:  ../8f0b88a8-e9b8-0fd8-4dd1-ad14c4fdca0e@darold.net/
Also on:    postgresql.org/message-id/8f0b88a8-e9b8-0fd8-4dd1-ad14c4fdca0e@darold.net

 · 

reply

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-hackers@postgresql.org
  Cc: gilles@darold.net, pgsql-hackers@lists.postgresql.org
  Subject: Re: Deparsing rewritten query
  In-Reply-To: <8f0b88a8-e9b8-0fd8-4dd1-ad14c4fdca0e@darold.net>

* 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