[SERVER-36079] WiredTigerKVEngineRepairTest should not move a file while it is being accessed Created: 11/Jul/18 Updated: 27/Oct/23 Resolved: 16/Jul/18 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Louis Williams | Assignee: | Louis Williams |
| Resolution: | Gone away | Votes: | 0 |
| Labels: | nyc | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Operating System: | ALL | ||||
| Sprint: | Storage NYC 2018-07-16 | ||||
| Participants: | |||||
| Linked BF Score: | 15 | ||||
| Description |
|
There are two problems with this test on Windows, both of which result in an error message: "The process cannot access the file because it is being used by another process" 1. The data file is renamed while the storage engine has an open handle on it. In this case, I think the file can be copied instead. 2. On line 220, for reasons unknown, the data file is still busy even after the ident is dropped. https://github.com/mongodb/mongo/blob/bccb252836954ac1963f8daf0d7bbdcf7af1d01c/src/mongo/db/storage/wiredtiger/wiredtiger_kv_engine_test.cpp#L220 this fix should also attempt to ensure the file has been delete, possibly by force deleting it. |
| Comments |
| Comment by Max Hirschhorn [ 13/Jul/18 ] |
We used the exclamation point in the task name to indicate it is a display task rather than an execution task. That is to say, the "unittests!" box represents the combined success/failure of the "compile_unittests" and "unittests" tasks. |
| Comment by Ian Whalen (Inactive) [ 13/Jul/18 ] |
|
Sorry, amending that - I misread the exclamation mark and this is not on a requred builder. max.hirschhorn - what does an exclamation mark in a task name mean? |
| Comment by Ian Whalen (Inactive) [ 13/Jul/18 ] |
|
100% failure on a required builder and Louis is out today. Should revert ASAP geert.bosch matthew.saltz |