8
16
Fork 22

Use futures and tokio to send activities #620

Zusammengeführt
elegaanz hat 4 Commits von async-federation nach master vor 5 Jahren zusammengeführt
elegaanz hat vor 5 Jahren kommentiert (Migriert von github.com)

Fixes #252

Publishing an article and having it federated to the whole fediverse only takes a few seconds now (instead of minutes before, on big Plume instances connected to many other ones).

Fixes #252 Publishing an article and having it federated to the whole fediverse only takes a few seconds now (instead of minutes before, on big Plume instances connected to many other ones).
igalic (Migriert von github.com) hat vor 5 Jahren überprüft
igalic (Migriert von github.com) hat einen Kommentar hinterlassen

👀

👀
igalic (Migriert von github.com) hat vor 5 Jahren kommentiert

is this comment still relevant?

is this comment still relevant?
igalic (Migriert von github.com) hat vor 5 Jahren kommentiert

if this fails, we panic with "new rt".??

if this fails, we panic with "new rt".??
igalic (Migriert von github.com) hat vor 5 Jahren kommentiert

i think we should talk about our logging strategy sometime 😅

i think we should talk about our logging strategy sometime 😅
elegaanz (Migriert von github.com) hat vor 5 Jahren überprüft
elegaanz (Migriert von github.com) hat vor 5 Jahren kommentiert

Not really, but it is for the background jobs in general, for the moment we just spawn them in a separate thread and that's it. I will remove it and create a more general issue.

Not really, but it is for the background jobs in general, for the moment we just spawn them in a separate thread and that's it. I will remove it and create a more general issue.
elegaanz (Migriert von github.com) hat vor 5 Jahren überprüft
elegaanz (Migriert von github.com) hat vor 5 Jahren kommentiert

Hum… yes, I could probably change that. To be honest I copied this line from reqwest examples without really thinking about it 😅

Hum… yes, I could probably change that. To be honest I copied this line from reqwest examples without really thinking about it :sweat_smile:
elegaanz (Migriert von github.com) hat vor 5 Jahren überprüft
elegaanz (Migriert von github.com) hat vor 5 Jahren kommentiert

Indeed, we should probably use log and env_logger or something like that. Should I create an issue?

Indeed, we should probably use `log` and `env_logger` or something like that. Should I create an issue?
codecov[bot] hat vor 5 Jahren kommentiert (Migriert von github.com)

Codecov Report

Merging #620 into master will increase coverage by 0.08%.
The diff coverage is 26.66%.

@@            Coverage Diff             @@
##           master     #620      +/-   ##
==========================================
+ Coverage   35.28%   35.37%   +0.08%     
==========================================
  Files          68       68              
  Lines        7907     7904       -3     
  Branches     1893     1887       -6     
==========================================
+ Hits         2790     2796       +6     
+ Misses       4345     4338       -7     
+ Partials      772      770       -2
# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/620?src=pr&el=h1) Report > Merging [#620](https://codecov.io/gh/Plume-org/Plume/pull/620?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/6dbc043a7e4d1757851599859683804bf36a3409?src=pr&el=desc) will **increase** coverage by `0.08%`. > The diff coverage is `26.66%`. ```diff @@ Coverage Diff @@ ## master #620 +/- ## ========================================== + Coverage 35.28% 35.37% +0.08% ========================================== Files 68 68 Lines 7907 7904 -3 Branches 1893 1887 -6 ========================================== + Hits 2790 2796 +6 + Misses 4345 4338 -7 + Partials 772 770 -2 ```
trinity-1686a hat vor 5 Jahren überprüft
trinity-1686a hat einen Kommentar hinterlassen
Besitzer

👀

👀
trinity-1686a hat vor 5 Jahren kommentiert
Besitzer

I think this print should be done in the map just under, otherwise output could get mixed quickly, making both logs useless

I think this print should be done in the map just under, otherwise output could get mixed quickly, making both logs useless
trinity-1686a hat vor 5 Jahren kommentiert
Besitzer

(what is that comma for?)

(what is that comma for?)
elegaanz (Migriert von github.com) hat vor 5 Jahren überprüft
elegaanz (Migriert von github.com) hat vor 5 Jahren kommentiert

(no idea, I will remove it)

(no idea, I will remove it)
elegaanz (Migriert von github.com) hat vor 5 Jahren überprüft
elegaanz (Migriert von github.com) hat vor 5 Jahren kommentiert

Actually, it is added by cargo-fmt, because this whole expression is the argument of a function.

Actually, it is added by cargo-fmt, because this whole expression is the argument of a function.
trinity-1686a hat vor 5 Jahren überprüft
trinity-1686a hat vor 5 Jahren kommentiert
Besitzer

okay...

okay...
trinity-1686a hat die Änderungen vor 5 Jahren genehmigt

Reviewer

trinity-1686a hat die Änderungen vor 5 Jahren genehmigt
Der Pull Request wurde als 7ea4acc289 gemergt.

Schritt 1:

Wechsle auf einen neuen Branch in deinem lokalen Repository und teste die Änderungen.
git checkout -b async-federation master
git pull origin async-federation

Schritt 2:

Führe die Änderungen zusammen und aktualisiere den Stand online auf Forgejo.
git checkout master
git merge --no-ff async-federation
git push origin master
Anmelden, um an der Diskussion teilzunehmen.
Keine Reviewer
Kein Meilenstein
Niemand zuständig
2 Beteiligte
Nachrichten
Fällig am
Das Fälligkeitsdatum ist ungültig oder außerhalb des zulässigen Bereichs. Bitte verwende das Format „jjjj-mm-tt“.

Kein Fälligkeitsdatum gesetzt.

Abhängigkeiten

Keine Abhängigkeiten gesetzt.

Referenz: Plume/Plume#620
Laden…
Hier gibt es bis jetzt noch keinen Inhalt.