Return default snapshot count to 10,000.

The huge (100k+) value was justified when storev2 was being dumped completely with every snapshot.

With storev2 being decomissioned we can checkpoint more frequently for faster recovery.

Signed-off-by: James Blair <mail@jamesblair.net>

CHANGELOG/CHANGELOG-3.6.md

@@ -46,6 +46,7 @@ See [code changes](https://github.com/etcd-io/etcd/compare/v3.5.0...v3.6.0).
 
 ### Package `server`
 
+- Decreased [`--snapshot-count` default value from 100,000 to 10,000](https://github.com/etcd-io/etcd/pull/15408)
 - Package `mvcc` was moved to `storage/mvcc`
 - Package `mvcc/backend` was moved to `storage/backend`
 - Package `mvcc/buckets` was moved to `storage/schema`

server/etcdmain/help.go

@@ -54,7 +54,7 @@ Member:
     Path to the data directory.
   --wal-dir ''
     Path to the dedicated wal directory.
-  --snapshot-count '100000'
+  --snapshot-count '10000'
     Number of committed transactions to trigger a snapshot to disk.
   --heartbeat-interval '100'
     Time (in milliseconds) of a heartbeat interval.

server/etcdserver/server.go

@@ -74,7 +74,7 @@ import (
 )
 
 const (
-	DefaultSnapshotCount = 100000
+	DefaultSnapshotCount = 10000
 
 	// DefaultSnapshotCatchUpEntries is the number of entries for a slow follower
 	// to catch-up after compacting the raft storage entries.

tests/framework/e2e/cluster.go

@@ -148,7 +148,7 @@ type EtcdProcessClusterConfig struct {
 
 	MetricsURLScheme string
 
-	SnapshotCount          int // default is 100000
+	SnapshotCount          int // default is 10000
 	SnapshotCatchUpEntries int // default is 5000
 
 	Client        ClientConfig