NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit5257b9b

committed

Close the file descriptor in ApplyLogicalMappingFile

The function was forgetting to close the file descriptor, resultingin failures like this: ERROR: 53000: exceeded maxAllocatedDescs (492) while trying to open file "pg_logical/mappings/map-4000-4eb-1_60DE1E08-5376b5-537c6b" LOCATION: OpenTransientFile, fd.c:2161Simply close the file at the end, and backpatch to 9.4 (where logicaldecoding was introduced). While at it, fix a nearby typo.Discussion:https://www.postgresql.org/message-id/flat/738a590a-2ce5-9394-2bef-7b1caad89b37%402ndquadrant.com

1 parentc2a2e33 commit5257b9bCopy full SHA for 5257b9b

File tree

1 file changed

-1

lines changed

src/backend/replication/logical
- reorderbuffer.c

1 file changed

-1

lines changed

`‎src/backend/replication/logical/reorderbuffer.c‎`

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -3328,11 +3328,13 @@ ApplyLogicalMappingFile(HTAB tuplecid_data, Oid relid, const char fname)`
`3328`	`3328`	`new_ent->combocid=ent->combocid;`
`3329`	`3329`	`}`
`3330`	`3330`	`}`
	`3331`	`+`
	`3332`	`+CloseTransientFile(fd);`
`3331`	`3333`	`}`
`3332`	`3334`
`3333`	`3335`
`3334`	`3336`	`/*`
`3335`		`- * Check whether theTransactionOId 'xid' is in the pre-sorted array 'xip'.`
	`3337`	`+ * Check whether theTransactionOid 'xid' is in the pre-sorted array 'xip'.`
`3336`	`3338`	`*/`
`3337`	`3339`	`staticbool`
`3338`	`3340`	`TransactionIdInArray(TransactionIdxid,TransactionId*xip,Sizenum)`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit5257b9b

File tree

1 file changed

1 file changed

`‎src/backend/replication/logical/reorderbuffer.c‎`

0 commit comments