reduce reqwest timeout to 5s #557

Yhdistetty
Plume_migration_agent yhdistetty 1 committia lähteestä reduce-timeout kohteeseen master 5 vuotta sitten
Omistaja

Current timeout is 30s, this add a second kind of timeout : if connection to remote can't be established in 5s, the remote is considered down, so that a thread don't stall for too long for no reason (and also make some unit test much faster to run)

Current timeout is 30s, this add a second kind of timeout : if connection to remote can't be established in 5s, the remote is considered down, so that a thread don't stall for too long for no reason (and also make some unit test much faster to run)
elegaanz (Migrated from github.com) hyväksyi nämä muutokset 5 vuotta sitten
elegaanz (Migrated from github.com) jätti kommentin

It makes me think: couldn't we have a unique client (in a lazy_static! or something like that). I think reqwest keep connections for a certain time if the client is not dropped, and it probably caches DNS too.

Otherwise, it looks good to me, so if the CI is happy you can merge.

It makes me think: couldn't we have a unique client (in a `lazy_static!` or something like that). I think reqwest keep connections for a certain time if the client is not dropped, and it probably caches DNS too. Otherwise, it looks good to me, so if the CI is happy you can merge.
codecov[bot] kommentoi 5 vuotta sitten (Migrated from github.com)

Codecov Report

Merging #557 into master will increase coverage by 2.71%.
The diff coverage is 62.96%.

@@            Coverage Diff             @@
##           master     #557      +/-   ##
==========================================
+ Coverage   40.03%   42.75%   +2.71%     
==========================================
  Files          69       70       +1     
  Lines        8035     8846     +811     
==========================================
+ Hits         3217     3782     +565     
- Misses       4818     5064     +246
# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/557?src=pr&el=h1) Report > Merging [#557](https://codecov.io/gh/Plume-org/Plume/pull/557?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/8f1ab3485e3f70072fa3bf276dab31c1c6bc50bd?src=pr&el=desc) will **increase** coverage by `2.71%`. > The diff coverage is `62.96%`. ```diff @@ Coverage Diff @@ ## master #557 +/- ## ========================================== + Coverage 40.03% 42.75% +2.71% ========================================== Files 69 70 +1 Lines 8035 8846 +811 ========================================== + Hits 3217 3782 +565 - Misses 4818 5064 +246 ```
Tekijä
Omistaja

maybe not a unique client (if we have multiple request to do at the same time they will be queued), but we could probably make some kind of pool of clients. I'll keep the idea somewhere for when I have spare time to spend

maybe not a unique client (if we have multiple request to do at the same time they will be queued), but we could probably make some kind of pool of clients. I'll keep the idea somewhere for when I have spare time to spend

Reviewers

The pull request has been merged as 33a0c7dcd3.
You can also view command line instructions.

Vaihe 1:

From your project repository, check out a new branch and test the changes.
git checkout -b reduce-timeout master
git pull origin reduce-timeout

Vaihe 2:

Merge the changes and update on Forgejo.
git checkout master
git merge --no-ff reduce-timeout
git push origin master
Sign in to join this conversation.
No reviewers
Ei merkkipaalua
Ei käsittelijää
2 osallistujaa
Ilmoitukset
Määräpäivä
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

Määräpäivää ei asetettu.

Riippuvuudet

Riippuvuuksia ei asetettu.

Reference: Plume/Plume#557
Ladataan…
Sisältöä ei vielä ole.