dolysis
/
rocksdb
mirror of https://github.com/facebook/rocksdb.git
3
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix TransactionTest::SeqAdvanceTest ASAN failure 

Summary:
The test didn't delete txn before creating a new one.
Closes https://github.com/facebook/rocksdb/pull/2913

Differential Revision: D5880236

Pulled By: yiwu-arbug

fbshipit-source-id: 7a4fcaada3d86332292754502cd8f4341143bf4f
This commit is contained in:
Yi Wu 2017-09-21 09:47:14 -07:00 committed by Facebook Github Bot
parent 3fc08fa88e
commit 1480e6f7cf
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
utilities/transactions/transaction_test.cc
View File

@ -4703,6 +4703,7 @@ TEST_P(TransactionTest, SeqAdvanceTest) {
ASSERT_OK(s); ASSERT_OK(s);
seq = db_impl->GetLatestSequenceNumber(); seq = db_impl->GetLatestSequenceNumber();
ASSERT_EQ(exp_seq, seq); ASSERT_EQ(exp_seq, seq);
delete txn;
// Commit without prepare. It shoudl write to DB without a commit marker. // Commit without prepare. It shoudl write to DB without a commit marker.
txn = db->BeginTransaction(write_options, txn_options); txn = db->BeginTransaction(write_options, txn_options);