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

movr: update workload to use random values in columns rather than NULL #42483

Merged
merged 1 commit into from
Nov 14, 2019

Conversation

rohany
Copy link
Contributor

@rohany rohany commented Nov 14, 2019

The original pass at the movr workload filled up the non-indexed columns with
null values rather than using randomly generated values. This PR fixes that.

Fixes #42435.

Release note (bug fix): The movr workload now populates table columns with randomly
generated data instead of NULLs.

The original pass at the movr workload filled up the non-indexed columns with
null values rather than using randomly generated values. This PR fixes that.

Fixes cockroachdb#42435.

Release note (bug fix): The movr workload now populates table columns with randomly
generated data instead of NULLs.
@rohany rohany requested a review from danhhz November 14, 2019 16:07
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@rohany
Copy link
Contributor Author

rohany commented Nov 14, 2019

bors r+

craig bot pushed a commit that referenced this pull request Nov 14, 2019
42483: movr: update workload to use random values in columns rather than NULL r=rohany a=rohany

The original pass at the movr workload filled up the non-indexed columns with
null values rather than using randomly generated values. This PR fixes that.

Fixes #42435.

Release note (bug fix): The movr workload now populates table columns with randomly
generated data instead of NULLs.

Co-authored-by: Rohan Yadav <rohany@alumni.cmu.edu>
@craig
Copy link
Contributor

craig bot commented Nov 14, 2019

Build succeeded

@craig craig bot merged commit 7e28ca9 into cockroachdb:master Nov 14, 2019
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.

movr workload data generation problems
3 participants