Commit 3d5b43db authored by Donald H. (Donnie) Pinkston, III's avatar Donald H. (Donnie) Pinkston, III
Browse files

Update buffer-mgr concurrency test to close files

The Buffer Manager concurrency test wasn't closing the data files it
created, so they couldn't be deleted during the cleanup phase on
Windows.  This caused test failures on Windows but not on *nix.
parent 0ee9c83d
Showing with 8 additions and 1 deletion
+8 -1
......@@ -57,6 +57,9 @@ public class TestBufferManager extends StorageTestCase {
verify(fileMgr, times(1)).loadPage(file, 3, page3.getPageData(), true);
verify(fileMgr, times(1)).loadPage(file, 5, page.getPageData(), true);
bufMgr.removeDBFile(file);
fileMgr.closeDBFile(file);
}
......@@ -93,7 +96,7 @@ public class TestBufferManager extends StorageTestCase {
}
// Give the test 60 seconds to complete.
Concurrent.assertConcurrent("Concurrent reads and writes", tasks, 60);
Concurrent.assertConcurrent("Concurrent reads and writes", tasks, 120);
}
......@@ -143,5 +146,9 @@ public class TestBufferManager extends StorageTestCase {
page.unpin();
}
// Close the data file.
bufMgr.removeDBFile(file);
fileMgr.closeDBFile(file);
}
}
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment