search

canonical / dqlite

Embeddable, replicated and fault-tolerant SQL engine.
https://dqlite.io
Other
3.9k stars 219 forks source link

Integrated thread pool, fixed all UTs and ITs #609

Closed just-now closed 8 months ago

just-now commented 9 months ago

WIP

MathieuBordere commented 9 months ago

We probably need some kind of integration branch for merging this kind of stuff into instead of going straight to master.

cole-miller commented 9 months ago

@MathieuBordere yeah agreed about the integration branch, we want to be able to cut releases that use the old code until the new stuff is ready for prime time

cole-miller commented 9 months ago

I've created the dqlite-next branch, we can use it as a target for PRs like this.

codecov[bot] commented 9 months ago

Codecov Report

Attention: Patch coverage is 80.80808% with 19 lines in your changes are missing coverage. Please review.

:exclamation: No coverage uploaded for pull request base (dqlite-next@498d627). Click here to learn what that means.

Files Patch % Lines
src/server.c 29.41% 11 Missing and 1 partial :warning:
src/lib/threadpool.c 72.72% 2 Missing and 4 partials :warning:
src/gateway.c 96.96% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## dqlite-next #609 +/- ## ============================================== Coverage ? 81.21% ============================================== Files ? 191 Lines ? 27096 Branches ? 4943 ============================================== Hits ? 22006 Misses ? 3462 Partials ? 1628 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.