Skip to content

storage: add Writer.PutRawMVCCRangeKey#84962

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
erikgrinaker:put-raw-mvcc-rangekey
Jul 26, 2022
Merged

storage: add Writer.PutRawMVCCRangeKey#84962
craig[bot] merged 1 commit intocockroachdb:masterfrom
erikgrinaker:put-raw-mvcc-rangekey

Conversation

@erikgrinaker
Copy link
Copy Markdown
Contributor

@erikgrinaker erikgrinaker commented Jul 24, 2022

This patch adds Writer.PutRawMVCCRangeKey(), which can be used to
write a range key with an already-encoded MVCCValue. This is useful to
avoid unnecessary decode/encode roundtrips when copying range keys.

Resolves #84896.

Release note: None

@erikgrinaker erikgrinaker requested a review from msbutler July 24, 2022 00:54
@erikgrinaker erikgrinaker self-assigned this Jul 24, 2022
@erikgrinaker erikgrinaker requested review from a team as code owners July 24, 2022 00:54
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

This patch adds `Writer.PutRawMVCCRangeKey()`, which can be used to
write a range key with an already-encoded `MVCCValue`. This is useful to
avoid unnecessary decode/encode roundtrips when copying range keys.

Release note: None
@erikgrinaker erikgrinaker force-pushed the put-raw-mvcc-rangekey branch from b13f90c to ff9f4d3 Compare July 24, 2022 00:55
Copy link
Copy Markdown
Contributor

@jbowens jbowens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 9 of 9 files at r1, all commit messages.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @msbutler)

@erikgrinaker
Copy link
Copy Markdown
Contributor Author

bors r=jbowens

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Jul 25, 2022

Build failed (retrying...):

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Jul 26, 2022

Build succeeded:

@craig craig bot merged commit 8db2669 into cockroachdb:master Jul 26, 2022
@erikgrinaker erikgrinaker deleted the put-raw-mvcc-rangekey branch July 26, 2022 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

storage: add Writer.PutRawMVCCRangeKey

3 participants