Commitd66b67f

committed

Allow background workers to connect to no particular database.

The documentation claims that this is supported, but it didn'tactually work. Fix that.Reported by Pavel Stehule; patch by me.

1 parent834aa56 commitd66b67fCopy full SHA for d66b67f

File tree

+13

-1

lines changed

+13

-1

lines changed

Lines changed: 13 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -779,7 +779,7 @@ InitPostgres(const char in_dbname, Oid dboid, const char username,`
`779`	`779`	`/* take database name from the caller, just for paranoia */`
`780`	`780`	`strlcpy(dbname,in_dbname,sizeof(dbname));`
`781`	`781`	`}`
`782`		`-else`
	`782`	`+elseif (OidIsValid(dboid))`
`783`	`783`	`{`
`784`	`784`	`/* caller specified database by OID */`
`785`	`785`	`HeapTupletuple;`
`@@ -799,6 +799,18 @@ InitPostgres(const char in_dbname, Oid dboid, const char username,`
`799`	`799`	`if (out_dbname)`
`800`	`800`	`strcpy(out_dbname,dbname);`
`801`	`801`	`}`
	`802`	`+else`
	`803`	`+{`
	`804`	`+/*`
	`805`	`+ * If this is a background worker not bound to any particular`
	`806`	`+ * database, we're done now. Everything that follows only makes`
	`807`	`+ * sense if we are bound to a specific database. We do need to`
	`808`	`+ * close the transaction we started before returning.`
	`809`	`+ */`
	`810`	`+if (!bootstrap)`
	`811`	`+CommitTransactionCommand();`
	`812`	`+return;`
	`813`	`+}`
`802`	`814`
`803`	`815`	`/*`
`804`	`816`	`* Now, take a writer's lock on the database we are trying to connect to.`

Comments

(0)