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

[core] limit number of client threads based on op count. #1323

Merged
merged 7 commits into from
Sep 7, 2019

Conversation

adally
Copy link
Contributor

@adally adally commented Jul 13, 2019

To solve issue #1176

brianfrankcooper#1212

The YCSB readme is as the following, which is a little misleading.Because we can't use '-p durability=false' to disable durability. Instead it worked when I used '-p durability=SKIP_WAL'. The readme should be written more clearly.

durability: Whether or not writes should be appended to the WAL. Bypassing the WAL can improve throughput but data cannot be recovered in the event of a crash. The default is true.
@busbey busbey changed the title update core.client.java [core] limit number of client threads based on op count. Aug 10, 2019
Copy link
Collaborator

@busbey busbey left a comment

Choose a reason for hiding this comment

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

This should use a logger or write to system.err, but we can fix that later.

@busbey busbey merged commit 2ff658c into brianfrankcooper:master Sep 7, 2019
@busbey busbey mentioned this pull request Sep 21, 2019
symat added a commit to symat/YCSB that referenced this pull request Jan 8, 2020
…kcooper#1382)

Previously, if you defined maxexecutiontime and not defined operationcount
parameter (or set operationcount=0), then you were able to run YCSB for a
given period of time and measure the number of operations you get during
the load test.

Since brianfrankcooper#1323 was merged, it it not possible to use operationcount=0, as
the thread count will be set to 0 in this case and no client thread will
be created. This PR makes it possible to use operationcount=0 again.
busbey pushed a commit that referenced this pull request Jan 31, 2020
Previously, if you defined maxexecutiontime and not defined operationcount
parameter (or set operationcount=0), then you were able to run YCSB for a
given period of time and measure the number of operations you get during
the load test.

Since #1323 was merged, it it not possible to use operationcount=0, as
the thread count will be set to 0 in this case and no client thread will
be created. This PR makes it possible to use operationcount=0 again.

fixes #1382
hse-project pushed a commit to hse-project/hse-ycsb that referenced this pull request Apr 17, 2020
…kcooper#1384)

Previously, if you defined maxexecutiontime and not defined operationcount
parameter (or set operationcount=0), then you were able to run YCSB for a
given period of time and measure the number of operations you get during
the load test.

Since brianfrankcooper#1323 was merged, it it not possible to use operationcount=0, as
the thread count will be set to 0 in this case and no client thread will
be created. This PR makes it possible to use operationcount=0 again.

fixes brianfrankcooper#1382
(cherry picked from commit 650eee2)
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.

2 participants