Commit 08af9f0d authored by Zardosht Kasheff's avatar Zardosht Kasheff Committed by Yoni Fogel

[t:4762], move comment

git-svn-id: file:///svn/toku/tokudb@42351 c7de825b-a66e-492c-adef-691d508d4ae1
parent 14b93c28
...@@ -250,13 +250,6 @@ garbage_collection(ULE ule, OMT snapshot_xids, OMT live_list_reverse, OMT live_r ...@@ -250,13 +250,6 @@ garbage_collection(ULE ule, OMT snapshot_xids, OMT live_list_reverse, OMT live_r
TXNID tl1; TXNID tl1;
TXNID xc = ule->uxrs[curr_committed_entry].xid; TXNID xc = ule->uxrs[curr_committed_entry].xid;
BOOL is_xc_live = toku_is_txn_in_live_root_txn_list(live_root_txns, xc);
if (is_xc_live) {
curr_committed_entry--;
continue;
}
tl1 = toku_get_youngest_live_list_txnid_for(xc, live_list_reverse);
// //
// If we find that the committed transaction is in the live list, // If we find that the committed transaction is in the live list,
// then xc is really in the process of being committed. It has not // then xc is really in the process of being committed. It has not
...@@ -266,6 +259,13 @@ garbage_collection(ULE ule, OMT snapshot_xids, OMT live_list_reverse, OMT live_r ...@@ -266,6 +259,13 @@ garbage_collection(ULE ule, OMT snapshot_xids, OMT live_list_reverse, OMT live_r
// As a result, we must mark what is just below xc as necessary and move on. // As a result, we must mark what is just below xc as necessary and move on.
// This issue was found while testing flusher threads, and was fixed for #3979 // This issue was found while testing flusher threads, and was fixed for #3979
// //
BOOL is_xc_live = toku_is_txn_in_live_root_txn_list(live_root_txns, xc);
if (is_xc_live) {
curr_committed_entry--;
continue;
}
tl1 = toku_get_youngest_live_list_txnid_for(xc, live_list_reverse);
if (tl1 == xc) { if (tl1 == xc) {
// if tl1 == xc, that means xc should be live and show up in // if tl1 == xc, that means xc should be live and show up in
// live_root_txns, which we check above. So, if we get // live_root_txns, which we check above. So, if we get
......
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