Skip to content

Commit

Permalink
Changes based on feedback of cmu-db#1349.
Browse files Browse the repository at this point in the history
  • Loading branch information
mbutrovich committed Jul 3, 2018
1 parent 40eff31 commit 8b7e092
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
6 changes: 4 additions & 2 deletions src/common/container/cuckoo_map.cpp
Expand Up @@ -14,7 +14,6 @@
#include <iostream>

#include "common/container/cuckoo_map.h"
#include "common/container/lock_free_queue.h"
#include "common/internal_types.h"
#include "common/item_pointer.h"
#include "common/logger.h"
Expand All @@ -33,6 +32,9 @@ class IndexMetric;

class StatementCache;

template <typename T>
class LockFreeQueue;

CUCKOO_MAP_TEMPLATE_ARGUMENTS
CUCKOO_MAP_TYPE::CuckooMap() {}

Expand Down Expand Up @@ -128,6 +130,6 @@ template class CuckooMap<StatementCache *, StatementCache *>;

// Used in TransactionLevelGCManager
template class CuckooMap<oid_t,
std::shared_ptr<peloton::LockFreeQueue<ItemPointer>>>;
std::shared_ptr<LockFreeQueue<ItemPointer>>>;

} // namespace peloton
2 changes: 1 addition & 1 deletion src/gc/transaction_level_gc_manager.cpp
Expand Up @@ -243,7 +243,7 @@ uint32_t TransactionLevelGCManager::Unlink(const uint32_t &thread_id,

if (txn_ctx->GetEpochId() <= expired_eid) {
// since this txn's epochId is <= the global expired epoch id
// no active transactions can read the version. Asa result,
// no active transactions can read the version. As a result,
// we can delete remove all of its garbage tuples from the indexes

// unlink versions from version chain and indexes
Expand Down

0 comments on commit 8b7e092

Please sign in to comment.