From: Bruce Momjian <bruce@momjian.us>
Date: Sat, 8 Mar 2014 17:20:30 +0000 (-0500)
Subject: C comments: improve description of relfilenode uniqueness
X-Git-Tag: REL9_4_BETA1~359
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=5024044a204ea4158c9223b28a31f60afc1e759f;p=postgresql

C comments: improve description of relfilenode uniqueness

Report by Antonin Houska
---

diff --git a/src/backend/catalog/catalog.c b/src/backend/catalog/catalog.c
index e3002eb963..282e0b9ae8 100644
--- a/src/backend/catalog/catalog.c
+++ b/src/backend/catalog/catalog.c
@@ -408,8 +408,8 @@ GetNewOidWithIndex(Relation relation, Oid indexId, AttrNumber oidcolumn)
 
 /*
  * GetNewRelFileNode
- *		Generate a new relfilenode number that is unique within the given
- *		tablespace.
+ *		Generate a new relfilenode number that is unique within the
+ *		database of the given tablespace.
  *
  * If the relfilenode will also be used as the relation's OID, pass the
  * opened pg_class catalog, and this routine will guarantee that the result
diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c
index 1de3170249..25f01e5165 100644
--- a/src/backend/commands/tablecmds.c
+++ b/src/backend/commands/tablecmds.c
@@ -9078,8 +9078,8 @@ ATExecSetTableSpace(Oid tableOid, Oid newTableSpace, LOCKMODE lockmode)
 	FlushRelationBuffers(rel);
 
 	/*
-	 * Relfilenodes are not unique across tablespaces, so we need to allocate
-	 * a new one in the new tablespace.
+	 * Relfilenodes are not unique in databases across tablespaces, so we
+	 * need to allocate a new one in the new tablespace.
 	 */
 	newrelfilenode = GetNewRelFileNode(newTableSpace, NULL,
 									   rel->rd_rel->relpersistence);
diff --git a/src/include/storage/relfilenode.h b/src/include/storage/relfilenode.h
index c995c10c4d..8616bd3a1e 100644
--- a/src/include/storage/relfilenode.h
+++ b/src/include/storage/relfilenode.h
@@ -55,7 +55,8 @@ typedef enum ForkNumber
  * relNode identifies the specific relation.  relNode corresponds to
  * pg_class.relfilenode (NOT pg_class.oid, because we need to be able
  * to assign new physical files to relations in some situations).
- * Notice that relNode is only unique within a particular tablespace.
+ * Notice that relNode is only unique within a database in a particular
+ * tablespace.
  *
  * Note: spcNode must be GLOBALTABLESPACE_OID if and only if dbNode is
  * zero.  We support shared relations only in the "global" tablespace.