aboutsummaryrefslogtreecommitdiff
path: root/src/backend/executor/nodeGather.c
diff options
context:
space:
mode:
authorMichael Paquier <michael@paquier.xyz>2020-07-16 15:52:37 +0900
committerMichael Paquier <michael@paquier.xyz>2020-07-16 15:52:37 +0900
commit932f9fb504a57f296cf698d15bd93462ddfe2776 (patch)
tree272232150da3375abfe6e67bececbf02334a68d1 /src/backend/executor/nodeGather.c
parented2c7f65bd9f15f8f7cd21ad61602f983b1e72e9 (diff)
downloadpostgresql-932f9fb504a57f296cf698d15bd93462ddfe2776.tar.gz
postgresql-932f9fb504a57f296cf698d15bd93462ddfe2776.zip
Switch pg_test_fsync to use binary mode on Windows
pg_test_fsync has always opened files using the text mode on Windows, as this is the default mode used if not enforced by _setmode(). This fixes a failure when running pg_test_fsync down to 12 because O_DSYNC and the text mode are not able to work together nicely. We fixed the handling of O_DSYNC in 12~ for the tool by switching to the concurrent-safe version of fopen() in src/port/ with 0ba06e0. And 40cfe86, by enforcing the text mode for compatibility reasons if O_TEXT or O_BINARY are not specified by the caller, broke pg_test_fsync. For all versions, this avoids any translation overhead, and pg_test_fsync should test binary writes, so it is a gain in all cases. Note that O_DSYNC is still not handled correctly in ~11, leading to pg_test_fsync to show insanely high numbers for open_datasync() (using this property it is easy to notice that the binary mode is much faster). This would require a backpatch of 0ba06e0 and 40cfe86, which could potentially break existing applications, so this is left out. There are no TAP tests for this tool yet, so I have checked all builds manually using MSVC. We could invent a new option to run a single transaction instead of using a duration of 1s to make the tests a maximum short, but this is left as future work. Thanks to Bruce Momjian for the discussion. Reported-by: Jeff Janes Author: Michael Paquier Discussion: https://postgr.es/m/16526-279ded30a230d275@postgresql.org Backpatch-through: 9.5
Diffstat (limited to 'src/backend/executor/nodeGather.c')
0 files changed, 0 insertions, 0 deletions