Skip to content

build: fix Pebble nightly benchmarks#74575

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
jbowens:jackson/pebble-benchmarks-fix
Jan 10, 2022
Merged

build: fix Pebble nightly benchmarks#74575
craig[bot] merged 1 commit intocockroachdb:masterfrom
jbowens:jackson/pebble-benchmarks-fix

Conversation

@jbowens
Copy link
Copy Markdown
Contributor

@jbowens jbowens commented Jan 7, 2022

The Pebble nightly benchmarks stopped running due to a build error
stemming from the removal of the generated code from the codebase.

Release note: None

The Pebble nightly benchmarks stopped running due to a build error
stemming from the removal of the generated code from the codebase.

Release note: None
@jbowens jbowens requested a review from a team as a code owner January 7, 2022 18:33
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@jbowens jbowens requested a review from a team January 7, 2022 18:47
Copy link
Copy Markdown
Contributor Author

@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.

@cockroachdb/dev-inf not sure if there's a more appropriate fix for this?

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained

Copy link
Copy Markdown
Collaborator

@nicktrav nicktrav 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 @jbowens)

@jbowens
Copy link
Copy Markdown
Contributor Author

jbowens commented Jan 10, 2022

TFTR!

bors r=nicktrav

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Jan 10, 2022

Build succeeded:

@craig craig bot merged commit e1d95fe into cockroachdb:master Jan 10, 2022
@jbowens jbowens deleted the jackson/pebble-benchmarks-fix branch January 12, 2022 18:21
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.

3 participants