Remove useless LZ4 system call on failure when writing file header
authorMichael Paquier <michael@paquier.xyz>
Wed, 24 Nov 2021 11:12:54 +0000 (20:12 +0900)
committerMichael Paquier <michael@paquier.xyz>
Wed, 24 Nov 2021 11:12:54 +0000 (20:12 +0900)
commitf79962d8264b8d205ce45a8aa11d1b37f9592a81
treeb6643cd8bb2e70047f7f7b7e9006cc616b21b87a
parent411137a429210e432f923264a8e313a9872910ca
Remove useless LZ4 system call on failure when writing file header

If an error occurs when writing the LZ4 file header, LZ4F_compressEnd()
was called in the error code path of write(), followed by
LZ4F_freeCompressionContext() to finish the cleanup.  The code as-is was
not broken, but the LZ4F_compressEnd() proves to not be necessary as
there are no contents to flush at this stage, so remove it.

Per gripe from Jeevan Ladhe and Robert Haas.

Discussion: https://postgr.es/m/CAOgcT0PE33wbD7giAT1OSkNJt=p-vu8huq++qh=ny9O=SCP5aA@mail.gmail.com
src/bin/pg_basebackup/walmethods.c