Open NULLx76 opened 5 years ago
Merging #52 into develop will not change coverage. The diff coverage is
80%
.
@@ Coverage Diff @@
## develop #52 +/- ##
========================================
Coverage 62.75% 62.75%
========================================
Files 23 23
Lines 1893 1893
Branches 507 507
========================================
Hits 1188 1188
Misses 226 226
Partials 479 479
Impacted Files | Coverage Δ | |
---|---|---|
src/crypto/mod.rs | 57.89% <ø> (ø) |
:arrow_up: |
src/serialization/mod.rs | 67.65% <100%> (ø) |
:arrow_up: |
src/crypto/signature.rs | 60.5% <66.66%> (ø) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 050fdbd...5550011. Read the comment docs.
Code Climate has analyzed commit 55500116 and detected 0 issues on this pull request.
View more on Code Climate.
I also want to benchmark our throughput when using a work stealing threadpool, we could look at how tokio benchmarks this and do something similar
I wonder why don't you use tokio
for reactor/socket/threadpool? Is tokio
too much for this? Or you wanted to minimize external dependencies?
We did indeed try tokio
but it was too much just like you said. The framework would "envelop" the application becoming comparable to twisted (requiring anyone that import the library to start it for example). We do use parts of tokio like mio which is basically the network backend of tokio, but we opted not to use the entire framework. The fact that it adds quite some external dependencies also factored in to this decision. We may reconsider using tokio when rust implements its own async which will happen in the not too distant future and also lessen the dependencies of tokio
considerably, or if/when we see another reason (like performance) to do so.
We'll likely run some benchmarks to compare performance.
Codecov Report
53.84% <ø> (ø)
57.89% <0%> (+1.57%)
81.08% <100%> (-1.68%)
67.65% <100%> (-2.62%)
70.83% <70.31%> (+18.2%)
60.5% <71.42%> (-3.01%)
71.42% <77.77%> (+0.15%)
57.08% <82.35%> (+1.94%)
Continue to review full report at Codecov.