Skip to content

Commit

Permalink
Merge #99992
Browse files Browse the repository at this point in the history
99992: kvserver: disable kv.expiration_leases_only.enabled metamorphism r=irfansharif a=erikgrinaker

This can prevent closed timestamp updates, causing test flakes.

Touches #99812.

Epic: none
Release note: None

Co-authored-by: Erik Grinaker <[email protected]>
  • Loading branch information
craig[bot] and erikgrinaker committed Mar 30, 2023
2 parents 645d0d3 + ea5dce3 commit 47bd780
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions pkg/kv/kvserver/replica_range_lease.go
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@ import (
"github.com/cockroachdb/cockroach/pkg/kv/kvserver/raftutil"
"github.com/cockroachdb/cockroach/pkg/roachpb"
"github.com/cockroachdb/cockroach/pkg/settings"
"github.com/cockroachdb/cockroach/pkg/util"
"github.com/cockroachdb/cockroach/pkg/util/hlc"
"github.com/cockroachdb/cockroach/pkg/util/log"
"github.com/cockroachdb/cockroach/pkg/util/retry"
Expand All @@ -80,7 +79,10 @@ var ExpirationLeasesOnly = settings.RegisterBoolSetting(
settings.SystemOnly,
"kv.expiration_leases_only.enabled",
"only use expiration-based leases, never epoch-based ones (experimental, affects performance)",
util.ConstantWithMetamorphicTestBool("kv.expiration_leases_only.enabled", false),
false,
// TODO(erikgrinaker): Make this metamorphic when they don't prevent closed
// timestamp updates: https://github.com/cockroachdb/cockroach/issues/99812
//util.ConstantWithMetamorphicTestBool("kv.expiration_leases_only.enabled", false),
)

var leaseStatusLogLimiter = func() *log.EveryN {
Expand Down

0 comments on commit 47bd780

Please sign in to comment.