}
}
-/*
- * Does LOCK TABLE work on non-table relations on this server?
- *
- * Note: assumes it is called out of any transaction
- */
-bool
-IsLockTableGeneric(Archive *AHX)
-{
- ArchiveHandle *AH = (ArchiveHandle *) AHX;
- PGresult *res;
- char *sqlstate;
- bool retval;
-
- if (AHX->remoteVersion >= 140000)
- return true;
- else if (AHX->remoteVersion < 90500)
- return false;
-
- StartTransaction(AHX);
-
- /*
- * Try a LOCK TABLE on a well-known non-table catalog; WRONG_OBJECT_TYPE
- * tells us that this server doesn't support locking non-table rels, while
- * LOCK_NOT_AVAILABLE and INSUFFICIENT_PRIVILEGE tell us that it does.
- * Report anything else as a fatal problem.
- */
-#define ERRCODE_INSUFFICIENT_PRIVILEGE "42501"
-#define ERRCODE_WRONG_OBJECT_TYPE "42809"
-#define ERRCODE_LOCK_NOT_AVAILABLE "55P03"
- res = PQexec(AH->connection,
- "LOCK TABLE pg_catalog.pg_class_tblspc_relfilenode_index IN ACCESS SHARE MODE NOWAIT");
- switch (PQresultStatus(res))
- {
- case PGRES_COMMAND_OK:
- retval = true;
- break;
- case PGRES_FATAL_ERROR:
- sqlstate = PQresultErrorField(res, PG_DIAG_SQLSTATE);
- if (sqlstate &&
- strcmp(sqlstate, ERRCODE_WRONG_OBJECT_TYPE) == 0)
- {
- retval = false;
- break;
- }
- else if (sqlstate &&
- (strcmp(sqlstate, ERRCODE_LOCK_NOT_AVAILABLE) == 0 ||
- strcmp(sqlstate, ERRCODE_INSUFFICIENT_PRIVILEGE) == 0))
- {
- retval = true;
- break;
- }
- /* else, falls through */
- default:
- warn_or_exit_horribly(AH, "LOCK TABLE failed for \"%s\": %s",
- "pg_catalog.pg_class_tblspc_relfilenode_index",
- PQerrorMessage(AH->connection));
- retval = false; /* not reached */
- break;
- }
- PQclear(res);
-
- CommitTransaction(AHX);
-
- return retval;
-}
-
void
StartTransaction(Archive *AHX)
{
ExecuteSqlStatement(AH, "SET row_security = off");
}
- /* Detect whether LOCK TABLE can handle non-table relations */
- AH->hasGenericLockTable = IsLockTableGeneric(AH);
-
/*
* Start transaction-snapshot mode transaction to dump consistent data.
*/
* assume our lock on the child is enough to prevent schema
* alterations to parent tables.
*
- * We only need to lock the relation for certain components; see
- * pg_dump.h
+ * NOTE: it'd be kinda nice to lock other relations too, not only
+ * plain or partitioned tables, but the backend doesn't presently
+ * allow that.
*
- * On server versions that support it, we lock all relations not just
- * plain tables.
+ * We only need to lock the table for certain components; see
+ * pg_dump.h
*/
if (tblinfo[i].dobj.dump &&
- (fout->hasGenericLockTable ||
- tblinfo[i].relkind == RELKIND_PARTITIONED_TABLE ||
- tblinfo[i].relkind == RELKIND_RELATION) &&
+ (tblinfo[i].relkind == RELKIND_RELATION ||
+ tblinfo->relkind == RELKIND_PARTITIONED_TABLE) &&
(tblinfo[i].dobj.dump & DUMP_COMPONENTS_REQUIRING_LOCK))
{
resetPQExpBuffer(query);