Revert "Parallelize template rendering"
authorrinpatch <rinpatch@sdf.org>
Tue, 10 Sep 2019 19:01:45 +0000 (22:01 +0300)
committerrinpatch <rinpatch@sdf.org>
Tue, 10 Sep 2019 19:01:45 +0000 (22:01 +0300)
commit43f02dfe38547e07fb189aab78539af9e02302b3
treea7b424ab5b104aa5317f99846f7f48aa0278f12c
parentedbaf781763a05037c1567f6f1a3ac1d35d60de2
Revert "Parallelize template rendering"

This reverts commit 1ad71592adb47762287aec8c36d0fca565c38362.

Since it had no limit on the number on concurrent processes it OOM killed
instances while rendering hellthreads. When I tried introducing a
concurrency limit with Task.async_stream/manual folds it lead to about 3 times
worse performance on threads larger than 1000 activities (we are talking
30s vs 1.2 minutes), I think this is not worth the about 1.5 times
performance increase on smaller threads when using it.
lib/mix/tasks/pleroma/benchmark.ex
lib/pleroma/web/mastodon_api/views/status_view.ex
lib/pleroma/web/web.ex