aboutsummaryrefslogtreecommitdiff
path: root/src/backend/utils/adt/jsonb_util.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2014-06-12 18:59:06 -0400
committerTom Lane <tgl@sss.pgh.pa.us>2014-06-12 18:59:14 -0400
commit3ba1776c568e36423ee66892481b6d1703f4fbd8 (patch)
tree89fcfb9140364745e02ef59e6dfaf536cbc45053 /src/backend/utils/adt/jsonb_util.c
parentb37e574865f8b465a1c99a29dcdfcecf6eca32f5 (diff)
downloadpostgresql-3ba1776c568e36423ee66892481b6d1703f4fbd8.tar.gz
postgresql-3ba1776c568e36423ee66892481b6d1703f4fbd8.zip
Improve tuplestore's error messages for I/O failures.
We should report the errno when we get a failure from functions like BufFileWrite. "ERROR: write failed" is unreasonably taciturn for a case that's well within the realm of possibility; I've seen it a couple times in the buildfarm recently, in situations that were probably out-of-disk-space, but it'd be good to see the errno to confirm it. I think this code was originally written without assuming that the buffile.c functions would return useful errno; but most other callers *are* assuming that, and a quick look at the buffile code gives no reason to suppose otherwise. Also, a couple of the old messages were phrased on the assumption that a short read might indicate a logic bug in tuplestore itself; but that code's pretty well tested by now, so a filesystem-level problem seems much more likely.
Diffstat (limited to 'src/backend/utils/adt/jsonb_util.c')
0 files changed, 0 insertions, 0 deletions