NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit9e79ca3

committed

Remove special cases for ETXTBSY from new fsync'ing logic.

The argument that this is a sufficiently-expected case to be silentlyignored seems pretty thin. Andres had brought it up back when we werestill considering that most fsync failures should be hard errors, and itprobably would be legit not to fail hard for ETXTBSY --- but the same istrue for EROFS and other cases, which is why we gave up on hard failures.ETXTBSY is surely not a normal case, so logging the failure seems finefrom here.

1 parentcb86785 commit9e79ca3Copy full SHA for 9e79ca3

File tree

1 file changed

-12

lines changed

src/backend/storage/file
- fd.c

1 file changed

-12

lines changed

`‎src/backend/storage/file/fd.c`

Lines changed: 3 additions & 12 deletions

Original file line number	Diff line number	Diff line change
`@@ -2418,18 +2418,15 @@ pre_sync_fname(const char *fname, bool isdir, int elevel)`
`2418`	`2418`	`{`
`2419`	`2419`	`if (errno==EACCES\|\| (isdir&&errno==EISDIR))`
`2420`	`2420`	`return;`
`2421`		`-`
`2422`		`-#ifdefETXTBSY`
`2423`		`-if (errno==ETXTBSY)`
`2424`		`-return;`
`2425`		`-#endif`
`2426`		`-`
`2427`	`2421`	`ereport(elevel,`
`2428`	`2422`	`(errcode_for_file_access(),`
`2429`	`2423`	`errmsg("could not open file \"%s\": %m",fname)));`
`2430`	`2424`	`return;`
`2431`	`2425`	`}`
`2432`	`2426`
	`2427`	`+/*`
	`2428`	`+ * We ignore errors from pg_flush_data() because this is only a hint.`
	`2429`	`+ */`
`2433`	`2430`	`(void)pg_flush_data(fd,0,0);`
`2434`	`2431`
`2435`	`2432`	`(void)close(fd);`
`@@ -2473,12 +2470,6 @@ fsync_fname_ext(const char *fname, bool isdir, int elevel)`
`2473`	`2470`	`{`
`2474`	`2471`	`if (errno==EACCES\|\| (isdir&&errno==EISDIR))`
`2475`	`2472`	`return;`
`2476`		`-`
`2477`		`-#ifdefETXTBSY`
`2478`		`-if (errno==ETXTBSY)`
`2479`		`-return;`
`2480`		`-#endif`
`2481`		`-`
`2482`	`2473`	`ereport(elevel,`
`2483`	`2474`	`(errcode_for_file_access(),`
`2484`	`2475`	`errmsg("could not open file \"%s\": %m",fname)));`

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!

Commit9e79ca3

File tree

1 file changed

1 file changed

`‎src/backend/storage/file/fd.c`

0 commit comments