Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

workload: fixture create can make table subset #23433

Merged
merged 1 commit into from
Mar 6, 2018

Conversation

jordanlewis
Copy link
Member

The new --only-tables flag allows specifying a subset of the tables in a
workload for creation. This is useful for benchmarking the workload
program itself, as well as creating partial backups of very large
datasets.

Release note: None

@jordanlewis jordanlewis requested a review from danhhz March 5, 2018 22:33
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@danhhz
Copy link
Contributor

danhhz commented Mar 5, 2018

:lgtm:


Review status: 0 of 2 files reviewed at latest revision, all discussions resolved, some commit checks pending.


pkg/workload/workload.go, line 154 at r1 (raw file):

}

func NewFilteringGenerator(gen Generator, filter map[string]struct{}) Generator {

sorry for the nit, but can we keep this in cmd/workload instead? or at least another file in this package, i'm trying to keep this file from bloating too much


Comments from Reviewable

The new --only-tables flag allows specifying a subset of the tables in a
workload for creation. This is useful for benchmarking the workload
program itself, as well as creating partial backups of very large
datasets.

Release note: None
@jordanlewis
Copy link
Member Author

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion.


pkg/workload/workload.go, line 154 at r1 (raw file):

Previously, danhhz (Daniel Harrison) wrote…

sorry for the nit, but can we keep this in cmd/workload instead? or at least another file in this package, i'm trying to keep this file from bloating too much

No problem.


Comments from Reviewable

@jordanlewis jordanlewis merged commit d7f2636 into cockroachdb:master Mar 6, 2018
@jordanlewis jordanlewis deleted the only-tables branch March 6, 2018 04:14
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