search

hyrise / hyrise-v1

HYRISE In-Memory Hybrid Storage Engine (archived, now developed in hyrise/hyrise repo)
https://github.com/hyrise/hyrise
MIT License
89 stars 44 forks source link

Fix issue #421: Test BufferedLoggerTests.simple_checkpoint_test fails #425

Closed Bensk1 closed 8 years ago

Bensk1 commented 8 years ago

Fixes #421 by clearing the storage manager during the SetUp of every test. It also removes calls to clear the StorageManager which are now unnecessary.

The test calls the recoverTables method of the storage manager. This method will recover all tables, including their indexes, whose names were found in the tableDumpDir. Therefore, it will also recover indexes which are still existing. This leads to failing.

Before executing recoverTables the test creates folders with names of tables which were used in previous tests by executing the Checkpoint plan operation. It creates folders for all table names, also table names used in previous tests, which are returned by the getTableNames method of the StorageManager.

mrks commented 8 years ago

(As the test suite uses ./test/persistency, not ./persistency, regular data is kept.)