[CLJ-823] Piping seque into seque can deadlock Created: 03/Aug/11 Updated: 29/Aug/14 Resolved: 29/Aug/14
|Affects Version/s:||Release 1.3|
|Fix Version/s:||Release 1.7|
Windows 7; JVM 1.6; Clojure 1.3 beta 1
|Patch:||Code and Test|
I'm not sure if this is a supported scenario, but the following deadlocks in Clojure 1.3:
Cause: As I understand it, the problem is that ys' fill takes place on an agent thread, so when it calls xs' drain, the (send-off agt fill) does not immediately trigger xs' fill, but is instead put on the nested list to be performed when ys' agent returns. Unfortunately, ys' fill will eventually block trying to take from xs, and so it never returns and the pending send-offs are never sent.
Approach: Use (release-pending-sends) in seque's drain function to avoid the deadlock when a seque is fed into another seque.
Screened by: Alex Miller
|Comment by Peter Monks [ 07/Jan/13 3:43 PM ]|
Reproduced on 1.4.0 and 1.5.0-RC1 as well, albeit with this example:
(seque 3 (seque 3 (range 10)))
|Comment by Stuart Halloway [ 30/Mar/13 9:16 AM ]|
|Comment by Andy Fingerhut [ 21/Aug/14 6:13 PM ]|
clj-823-v1.patch uses (release-pending-sends) in seque's drain function in an attempt to avoid the deadlock when a seque is fed into another seque, as suggested by Stuart Halloway. It adds Peter Monks's small quick test case demonstrating the deadlock, which fails (i.e. hangs until killed) without the change and passes with it.