]> granicus.if.org Git - postgresql/commit
Close the file descriptor in ApplyLogicalMappingFile
authorTomas Vondra <tomas.vondra@postgresql.org>
Thu, 16 Aug 2018 14:49:10 +0000 (16:49 +0200)
committerTomas Vondra <tomas.vondra@postgresql.org>
Thu, 16 Aug 2018 14:51:46 +0000 (16:51 +0200)
commit864ecd716a0dd191e587dd09e24992be5e720b26
tree5fd941179220d28d5d397b211d31d91b69b3a1b1
parent8e9f229d2bf668e780ce10d511925b11855afd2c
Close the file descriptor in ApplyLogicalMappingFile

The function was forgetting to close the file descriptor, resulting
in 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:2161

Simply close the file at the end, and backpatch to 9.4 (where logical
decoding was introduced). While at it, fix a nearby typo.

Discussion: https://www.postgresql.org/message-id/flat/738a590a-2ce5-9394-2bef-7b1caad89b37%402ndquadrant.com
src/backend/replication/logical/reorderbuffer.c