summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Lane2005-04-12 19:29:24 +0000
committerTom Lane2005-04-12 19:29:24 +0000
commitdffab0c88d17d4169b0827b0e5c7a8f7a4e8ebbb (patch)
tree1799d268d5791b4d10d6434bd51ff76c89967b07
parentc580d945415e93294d9b67f283a1d2a5a773b609 (diff)
Remove unnecessary UPDATE commands to assign explicit ACLs to functions
and PL languages during initdb. The default permissions for these objects are the same as what we were assigning anyway, so there is no need to expend space in the catalogs on them. The space cost is particularly significant in pg_proc's indexes, which are bloated by about a factor of 2 by the full-table update, and can never really recover the space. initdb not forced, since the change has no actual impact on behavior.
-rw-r--r--src/bin/initdb/initdb.c13
1 files changed, 5 insertions, 8 deletions
diff --git a/src/bin/initdb/initdb.c b/src/bin/initdb/initdb.c
index feaa54d9ea1..ff524d57909 100644
--- a/src/bin/initdb/initdb.c
+++ b/src/bin/initdb/initdb.c
@@ -39,7 +39,7 @@
* Portions Copyright (c) 1994, Regents of the University of California
* Portions taken from FreeBSD.
*
- * $PostgreSQL: pgsql/src/bin/initdb/initdb.c,v 1.80 2005/04/12 14:19:43 petere Exp $
+ * $PostgreSQL: pgsql/src/bin/initdb/initdb.c,v 1.81 2005/04/12 19:29:24 tgl Exp $
*
*-------------------------------------------------------------------------
*/
@@ -1672,7 +1672,10 @@ setup_conversion(void)
/*
* Set up privileges
*
- * We set most system catalogs and built-in functions as world-accessible.
+ * We mark most system catalogs as world-readable. We don't currently have
+ * to touch functions, languages, or databases, because their default
+ * permissions are OK.
+ *
* Some objects may require different permissions by default, so we
* make sure we don't overwrite privilege sets that have already been
* set (NOT NULL).
@@ -1687,12 +1690,6 @@ setup_privileges(void)
"UPDATE pg_class "
" SET relacl = '{\"=r/\\\\\"$POSTGRES_SUPERUSERNAME\\\\\"\"}' "
" WHERE relkind IN ('r', 'v', 'S') AND relacl IS NULL;\n",
- "UPDATE pg_proc "
- " SET proacl = '{\"=X/\\\\\"$POSTGRES_SUPERUSERNAME\\\\\"\"}' "
- " WHERE proacl IS NULL;\n",
- "UPDATE pg_language "
- " SET lanacl = '{\"=U/\\\\\"$POSTGRES_SUPERUSERNAME\\\\\"\"}' "
- " WHERE lanpltrusted;\n",
"GRANT USAGE ON SCHEMA pg_catalog TO PUBLIC;\n",
"GRANT CREATE, USAGE ON SCHEMA public TO PUBLIC;\n",
NULL