storage: disable handleRaftReady concurrency

We discovered that this exacerbates (and makes harder to fix) problems discovered in cockroachdb#7600.
tbg · Jul 7, 2016 · 8c02f0c · 8c02f0c
1 parent ee996f9
commit 8c02f0c
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/storage/store.go b/storage/store.go
@@ -19,7 +19,6 @@ package storage
 import (
 	"bytes"
 	"fmt"
-	"runtime"
 	"sync"
 	"sync/atomic"
 	"time"
@@ -86,7 +85,9 @@ var changeTypeInternalToRaft = map[roachpb.ReplicaChangeType]raftpb.ConfChangeTy
 	roachpb.REMOVE_REPLICA: raftpb.ConfChangeRemoveNode,
 }
 
-var storeReplicaRaftReadyConcurrency = 2 * runtime.NumCPU()
+// TODO(tschottdorf): It's currently unsafe to set this to any other value due
+// to concurrency issues. See #7672 and the discussion within.
+const storeReplicaRaftReadyConcurrency = 1
 
 // TestStoreContext has some fields initialized with values relevant
 // in tests.