Andres Freund <and...@anarazel.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 -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers