about summary refs log tree commit homepage
path: root/lib/PublicInbox/WQWorker.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2021-05-25 22:20:01 +0000
committerEric Wong <e@80x24.org>2021-05-25 23:05:02 +0000
commit708b182a57373172f5523f3dc297659d58e03b58 (patch)
tree90d2a254577ce297f33f7988ccc6eceed252435c /lib/PublicInbox/WQWorker.pm
parent3060d78b4183f3e985fb7ff8864949de990f2610 (diff)
downloadpublic-inbox-708b182a57373172f5523f3dc297659d58e03b58.tar.gz
ipc: wq: handle >MAX_ARG_STRLEN && <EMSGSIZE case
WQWorkers are limited roughly to MAX_ARG_STRLEN (the kernel
limit of argv + environ) to avoid excessive memory growth.
Occasionally, we need to send larger messages via workqueues
that are too small to hit EMSGSIZE on the sender.

This fixes "lei q" when using HTTP(S) externals, since that
code path sends large Eml objects from lei_xsearch workers
directly to lei2mail WQ workers.
Diffstat (limited to 'lib/PublicInbox/WQWorker.pm')
-rw-r--r--lib/PublicInbox/WQWorker.pm2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/PublicInbox/WQWorker.pm b/lib/PublicInbox/WQWorker.pm
index 3636321e..f7aa61c5 100644
--- a/lib/PublicInbox/WQWorker.pm
+++ b/lib/PublicInbox/WQWorker.pm
@@ -23,7 +23,7 @@ sub event_step {
         my ($self) = @_;
         my $n;
         do {
-                $n = $self->{wq}->recv_and_run($self->{sock}, 4096 * 33);
+                $n = $self->{wq}->recv_and_run($self->{sock});
         } while ($n);
         return if !defined($n) && $! == EAGAIN; # likely
         warn "wq worker error: $!\n" if !defined($n) && $! != ECONNRESET;