release-23.1: sql,keys: disregard checks in *ZoneConfig*Batch #110337
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backport 2/2 commits from #109774.
/cc @cockroachdb/release
Previously, a bug occured where a transactional schema change,
ALTER RANGE default CONFIGURE ZONE...
, statement wouldproduce a CPut failure, even though both statements do take effect.
This was due to a guard in the code that blocked us from adding the
first zone config to the uncommitted cache, causing the expValues
being updated to the KV batch to be the same for both statements.
This patch addresses the issue by removing the check and allowing
for
default
and psuedotables (like system ranges) to be added tothe uncommitted cache.
Epic: none
Release note (bug fix): two
ALTER RANGE default CONFIGURE ZONE
statements on the same line no longer displays an error.
Fixes: #108253
Release justification: small bugfix & update to code