Clojurians Log v2

There is an outstanding issue about head holding of closed over values, but that’s not directly tied to timeouts

Can you explain more about what you’re seeing?

And are you using the latest version or something older? 1.5.640 fixed an issue with clearing alt handlers

https://clojure.atlassian.net/browse/ASYNC-234

In systems with callbacks in java, timers can often be a source of leaks, because the inversion of control plus timers being a global resource means a timer is a strong reference to whatever data until it expires, and core.async doesn't remove timers when they are not chosen by an alt, it just marks them as cancelled

That may be it. We'll rerun with latest version and check again

And async-234 is the one I remembered

I can't explain it succinctly and I don't have the heap dump at hand, if you'd like I could probably share it with you. When you analyze it with Eclipse MAT it points to the synchronized queue behind timers at the dominating object which holds all the references in the heap

I am perfectly happy to blame the issue laid out in async-234 for any leaks in a program that uses core.async without any more evidence (I reported it and wrote the patches)

At work we had a lot of trouble with memory leaks in our project that heavily uses core.async and I ended up reducing our direct usage of timeouts. This was all a year or two before I figured out async-234(reducing timer usage was a wild guess "what is global and could be holding a ref?!"), so hard to definitively say.

And yes, all our leaks seem to point to timers in alts

Doesn't reduce accomplish the same thing?

reduce’s fn is 2 args, I only need one

90% of the time it will be something like tap> or println

Reduce and ignore the accumulator? Basically copy run!'s definition

Yes, my question is why this doesn't exist in core.async?

Oh. Uhh, reasons? 🙃