Skip to content

workload/ycsb: mark --read-modify-write-in-txn flag as RuntimeOnly#107726

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
nvb:nvanbenschoten/rmwRuntimeOnly
Jul 27, 2023
Merged

workload/ycsb: mark --read-modify-write-in-txn flag as RuntimeOnly#107726
craig[bot] merged 1 commit intocockroachdb:masterfrom
nvb:nvanbenschoten/rmwRuntimeOnly

Conversation

@nvb
Copy link
Copy Markdown
Contributor

@nvb nvb commented Jul 27, 2023

The read-modify-write-in-txn flag was added to ycsb in #103117.

This commit marks the flag as RuntimeOnly so that it doesn't break IMPORT when a newer version of workload (who knows about the flag) is run against an older version of cockroach (who does not).

Epic: None
Release note: None

The `read-modify-write-in-txn` flag was added to ycsb in cockroachdb#103117.

This commit marks the flag as RuntimeOnly so that it doesn't break IMPORT
when a newer version of `workload` (who knows about the flag) is run against
an older version of `cockroach` (who does not).

Epic: None
Release note: None
@nvb nvb requested a review from ZhouXing19 July 27, 2023 16:40
@nvb nvb requested a review from a team as a code owner July 27, 2023 16:40
@nvb nvb requested review from smg260 and srosenberg and removed request for a team July 27, 2023 16:40
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Copy link
Copy Markdown
Collaborator

@michae2 michae2 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 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @smg260, @srosenberg, and @ZhouXing19)

Copy link
Copy Markdown
Contributor

@sean- sean- 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 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! 2 of 0 LGTMs obtained (waiting on @smg260, @srosenberg, and @ZhouXing19)

@nvb
Copy link
Copy Markdown
Contributor Author

nvb commented Jul 27, 2023

TFTR!

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Jul 27, 2023

Build succeeded:

@craig craig bot merged commit 9d6232e into cockroachdb:master Jul 27, 2023
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.

5 participants