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

Enable io thread only #460

Merged
merged 1 commit into from
Jan 21, 2020
Merged

Enable io thread only #460

merged 1 commit into from
Jan 21, 2020

Conversation

senyer
Copy link
Collaborator

@senyer senyer commented Jan 21, 2020

1.Enable this flag to enable io thread only or disable io thread only.
2.Benchmark result of enable io thread only or disable io thread only:

True(enable io thread only):

==========Benchmark Result================

API                          str.put()
t_count                      2
c_request                    10000
cost_avg                     995.5 ms
cost_min                     994 ms
cost_max                     997 ms
tps                          10.05 k/s
----------------------------------------------------------
t_count                      2
c_request                    100000
cost_avg                     4176.0 ms
cost_min                     4158 ms
cost_max                     4194 ms
tps                          23.95 k/s
===========================================

False(disable io thread only):

============Benchmark Result===================
API                          str.put()
t_count                      2
c_request                    10000
cost_avg                     1255.5 ms
cost_min                     1218 ms
cost_max                     1293 ms
tps                          7.96 k/s
-----------------------------------------------------------------
t_count                      2
c_request                    100000
cost_avg                     4895.5 ms
cost_min                     4888 ms
cost_max                     4903 ms
tps                          20.43 k/s
==============================================

@codecov
Copy link

codecov bot commented Jan 21, 2020

Codecov Report

Merging #460 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #460   +/-   ##
=========================================
  Coverage     51.37%   51.37%           
  Complexity      194      194           
=========================================
  Files            54       54           
  Lines          1234     1234           
  Branches        132      132           
=========================================
  Hits            634      634           
  Misses          486      486           
  Partials        114      114

@kairbon
Copy link
Collaborator

kairbon commented Jan 21, 2020

Thanks, LGTM

@jovany-wang jovany-wang merged commit eab8b6a into distkv-project:master Jan 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants