From 1192be196b3d0acca2e2dce5ffd5d12a924fdc5a Mon Sep 17 00:00:00 2001
From: Juan Batiz-Benet <juan@benet.ai>
Date: Sat, 6 Dec 2014 13:39:03 -0800
Subject: [PATCH] blockstore: Put checks Has first.
---
blocks/blockstore/blockstore.go | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/blocks/blockstore/blockstore.go b/blocks/blockstore/blockstore.go
index 68ccc7c7..3fe742ef 100644
--- a/blocks/blockstore/blockstore.go
+++ b/blocks/blockstore/blockstore.go
@@ -45,7 +45,13 @@ func (bs *blockstore) Get(k u.Key) (*blocks.Block, error) {
}
func (bs *blockstore) Put(block *blocks.Block) error {
- return bs.datastore.Put(block.Key().DsKey(), block.Data)
+ // Has is cheaper than
+ k := block.Key().DsKey()
+ exists, err := bs.datastore.Has(k)
+ if err != nil && exists {
+ return nil // already stored.
+ }
+ return bs.datastore.Put(k, block.Data)
}
func (bs *blockstore) Has(k u.Key) (bool, error) {
--
GitLab