From: | Bruce Momjian <maillist(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | jwieck(at)debis(dot)com |
Cc: | pgsql-hackers(at)postgreSQL(dot)org |
Subject: | Re: [HACKERS] rules ON SELECT |
Date: | 1998-10-02 01:41:55 |
Message-ID: | 199810020141.VAA22675@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> Hi,
>
> I'm currently thinking about multiple action and non-INSTEAD
> rules ON SELECT. I'm not sure what users might expect when
> they get fired.
>
> Well if a user types SELECT ... FROM tab and there are rules
> ON SELECT TO tab, then of course. But what about if the user
> issues an INSERT INTO x SELECT ... FROM tab or an UPDATE x
> SET col = tab.attr? In fact tab is scanned and returns data.
> Should the rule ON SELECT then be fired too?
>
> And what the hell is all that good for? Do we need other
> rules ON SELECT than those that build views (which we have
> now)? Tracing which data one user uses? Cannot be what rules
> are made for.
I would guess that SELECT ... FROM tab and INSERT INTO x SELECT ... FROM
tab would use the rules, but I can see where and UPDATE is using the
data from the table just like a SELECT, so I can see it would be
confusing for some if it DID do the rule in that case, and for others if
it DID NOT use the rule. That is a no-win situation, and usually means
that there is a bad interface design. However, in our case, it is just
a "much too powerful" interface design.
Please use your judgement. I can see that you are saying that there
really is no reason for SELECT rules except to create views. However,
if someone wants a computed column as part of a table, AND they want the
table name to be the same, isn't a SELECT rule the only way to do that.
They could of course rename the table, and create a view with the old
name. This may be the clearest way to do it.
I think if you make it consistent and/or print something to the user
when they try and do something strange, that usually keeps people happy.
If they understand what it is doing, the will usually accept it,
especially if it is a SUPERSET of SQL.
>
> If nobody votes against, I would only add some code checking
> that there is at max one INSTEAD SELECT rule that returns
> exactly the relations tuple structure ON SELECT (currently
> with CREATE TABLE, CREATE RULE someone can setup a situation
> that crashes the backend on SELECT). So SELECT rules are
> totally restricted to view building and nothing else.
>
> After that I'll tidy up the rewrite code (the work I've done
> screwed it up a little with nearly duplicate functions).
> Anything except for bug fixing is then delayed for 6.5.
>
> I still have in mind that we wanted to have views of UNIONS,
> DISTINCT views and some more. But since they require totally
> different semantics (the resulting plan must have something
> like a subselect of union in the case of an UPDATE...) this
> is far too much and has bad bad traps deep inside. We all
> don't want to fall into one during BETA.
Sounds good to me.
--
Bruce Momjian | http://www.op.net/~candle
maillist(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 1998-10-02 01:42:01 | Re: [HACKERS] Patch for libpgtcl |
Previous Message | Vadim Mikheev | 1998-10-02 01:20:25 | Re: [HACKERS] It sorta works, but I'm confused about locking |