Skip to content

Commit 42f0c58

Browse files
authored
Fix unlikely provider panic when requesting search permit (#5816)
1 parent acc6f92 commit 42f0c58

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

quickwit/quickwit-search/src/search_permit_provider.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -163,10 +163,10 @@ impl SearchPermitActor {
163163
permits.push(SearchPermitFuture(rx));
164164
}
165165
self.assign_available_permits();
166-
permit_sender
167-
.send(permits)
168-
// This is a request response pattern, so we can safely ignore the error.
169-
.expect("Receiver lives longer than sender");
166+
// The receiver could be dropped in the (unlikely) situation
167+
// where the future requesting these permits is cancelled before
168+
// this message is processed.
169+
let _ = permit_sender.send(permits);
170170
}
171171
SearchPermitMessage::UpdateMemory { memory_delta } => {
172172
if self.total_memory_allocated as i64 + memory_delta < 0 {

0 commit comments

Comments
 (0)