| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Andres Freund <andres(at)anarazel(dot)de> |
| Cc: | pgsql-hackers(at)postgresql(dot)org, Dimitri Fontaine <dimitri(at)2ndquadrant(dot)fr> |
| Subject: | Re: Command Triggers |
| Date: | 2012-02-27 23:30:36 |
| Message-ID: | 21501.1330385436@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Andres Freund <andres(at)anarazel(dot)de> writes:
> Sorry for letting this slide.
> Is it worth adding this bit to OpenIntoRel? Not sure if there is danger in
> allowing anyone to create shared tables
> /* In all cases disallow placing user relations in pg_global */
> if (tablespaceId == GLOBALTABLESPACE_OID)
> ereport(ERROR,
> (errcode(ERRCODE_INVALID_PARAMETER_VALUE),
> errmsg("only shared relations can be placed in pg_global
> tablespace")));
Ugh ... if that's currently allowed, we definitely need to fix it.
But I'm not sure OpenIntoRel is the right place. I'd have expected
the test to be at some lower level, like heap_create_with_catalog
or so.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andres Freund | 2012-02-27 23:32:26 | Re: Command Triggers, patch v11 |
| Previous Message | Andres Freund | 2012-02-27 23:26:05 | Re: Command Triggers |