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 #422

Closed Bensk1 closed 8 years ago

Bensk1 commented 8 years ago

Fixes #421 by clearing the storage manager before executing the tests.

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

Could you list the steps needed to reproduce the issue? On a current hyrise master, I am not having any issues.

Bensk1 commented 8 years ago

Did you specify PERSISTENCY := BUFFEREDLOGGER in settings.mk?