peloton icon indicating copy to clipboard operation
peloton copied to clipboard

Published 20 hours ago • verified publisher icon cmu-db

Checkpointing

Open db-ol opened this issue 6 years ago • 6 comments

Checkpointing as part of the resistance mechanism.

db-ol avatar Apr 18 '18 23:04 db-ol

Coverage Status

Coverage increased (+0.4%) to 76.96% when pulling af92a667cabf21a41755a96f36c043f331e2fdbd on aaron-tian:checkpoint into 898219f9fe6e1d2b6901ea3797caac5c93850213 on cmu-db:master.

coveralls avatar Apr 19 '18 01:04 coveralls

@ksaito7 Tests are failing due to newly added files (e. g. -Werror=maybe-uninitialized). Could you please take a look at them? Thanks.

db-ol avatar Apr 19 '18 15:04 db-ol

Some modifications are necessary due to catalog::LayoutCatalog and storage::Layout added in current master branch.

ksaito7 avatar May 15 '18 05:05 ksaito7

  1. Change for catalog::SystemCatalogs
  • Use SystemCatalogs in catalog checkpointing and recovery in addition to catalogs in catalog DB.
  • Eliminate basic catalogs in catalog DB from checkpoint target to avoid a table collision when a catalog table changes.
  • Add bootstrapping of SystemCatalogs when a database storage object is recovered.
  1. Change for catalog::SchemaCatalog
  • Add SchemaCatalog checkpointing and recovery
  • Fix to use schema name when getting TableCatalogObject
  • Use CATALOG_SCHEMA_NAME to distinguish catalog tables from user tables .
  1. Chagen for catalog::LayoutCatalog and storage::Layout
  • Add LayoutCatalog checkpointing and recovery
  • Add layout serialize/deserialize function for tile groups recovery

ksaito7 avatar May 16 '18 23:05 ksaito7

AddressSanitizer error on only release build in travis was resolved by deleting DataTable GetInfo function call in recovery test. I didn't know eventually why this error was occurred because AddressSanitizer didn't output the place. But it is resolved, so please review our codes again.

ksaito7 avatar May 23 '18 16:05 ksaito7

Update for PR #1383

ksaito7 avatar Jun 21 '18 00:06 ksaito7