@nxtedition/rocksdb 10.0.6 → 10.0.9

package/binding.cc

@@ -323,7 +323,7 @@ struct BaseIterator : public Closable {
 
   virtual ~BaseIterator() { assert(!iterator_); }
 
-  bool DidSeek() const { return iterator_ != nullptr; }
+  bool DidSeek() const { return iterator_ != nullptr && !target_; }
 
   void SeekToRange() {
     if (!iterator_) {
@@ -338,23 +338,19 @@ struct BaseIterator : public Closable {
   }
 
   void Seek(const rocksdb::Slice& target) {
-    if (!iterator_) {
-      Init();
-    }
-
-    if ((upper_bound_ && target.compare(*upper_bound_) >= 0) || (lower_bound_ && target.compare(*lower_bound_) < 0)) {
-      // TODO (fix): Why is this required? Seek should handle it?
-      // https://github.com/facebook/rocksdb/issues/9904
-      iterator_->SeekToLast();
-      if (iterator_->Valid()) {
-        iterator_->Next();
-      }
-    } else if (reverse_) {
-      iterator_->SeekForPrev(target);
-    } else {
-      iterator_->Seek(target);
-    }
-  }
+		if ((upper_bound_ && target.compare(*upper_bound_) >= 0) || (lower_bound_ && target.compare(*lower_bound_) < 0)) {
+			// TODO (fix): Why is this required? Seek should handle it?
+			// https://github.com/facebook/rocksdb/issues/9904
+			iterator_->SeekToLast();
+			if (iterator_->Valid()) {
+				iterator_->Next();
+			}
+		} else if (reverse_) {
+			iterator_->SeekForPrev(target);
+		} else {
+			iterator_->Seek(target);
+		}
+	}
 
   rocksdb::Status Close() override {
     snapshot_.reset();
@@ -422,6 +418,7 @@ struct BaseIterator : public Closable {
   std::optional<rocksdb::PinnableSlice> lower_bound_;
   std::optional<rocksdb::PinnableSlice> upper_bound_;
   std::unique_ptr<rocksdb::Iterator> iterator_;
+  std::optional<std::string> target_;
   const bool reverse_;
   const int limit_;
   const bool fillCache_;
@@ -1275,7 +1272,7 @@ NAPI_METHOD(iterator_seek) {
   NAPI_STATUS_THROWS(GetValue(env, argv[1], target));
 
   iterator->first_ = true;
-  iterator->Seek(target);  // TODO: Does seek causing blocking IO?
+  iterator->Seek(target);
 
   return 0;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nxtedition/rocksdb",
-  "version": "10.0.6",
+  "version": "10.0.9",
   "description": "A low-level Node.js RocksDB binding",
   "license": "MIT",
   "main": "index.js",