From cfaa33f9a5de1c1d80b213cfe8fffcc1dd50168a Mon Sep 17 00:00:00 2001 From: agiardullo Date: Mon, 19 Oct 2015 12:40:15 -0700 Subject: [PATCH] Update transaction iterator documentation Summary: Remove warning about an issue that was resolved. Turns out the issue was a false-alarm. Test Plan: n/a Reviewers: igor, yhchiang, rven, sdong Reviewed By: sdong Subscribers: dhruba, leveldb Differential Revision: https://reviews.facebook.net/D49011 --- include/rocksdb/utilities/transaction.h | 4 ---- 1 file changed, 4 deletions(-) diff --git a/include/rocksdb/utilities/transaction.h b/include/rocksdb/utilities/transaction.h index bd5589b73..cbeeceabc 100644 --- a/include/rocksdb/utilities/transaction.h +++ b/include/rocksdb/utilities/transaction.h @@ -214,10 +214,6 @@ class Transaction { // // The returned iterator is only valid until Commit(), Rollback(), or // RollbackToSavePoint() is called. - // NOTE: Transaction::Put/Merge/Delete will currently invalidate this iterator - // until - // the following issue is fixed: - // https://github.com/facebook/rocksdb/issues/616 virtual Iterator* GetIterator(const ReadOptions& read_options) = 0; virtual Iterator* GetIterator(const ReadOptions& read_options,