core-async

2020-12-03T18:35:17.296100Z

@alexmiller based on a discussion in #clojure I took another look at https://clojure.atlassian.net/browse/ASYNC-225 and decided their root cause analysis didn't go far enough, so their conclusion (they need a new feature, non-shared timeouts) was faulty. I dropped a patch on the ticket that I think solves their real issue (the non-selected channel operations in an alt retain a reference to closed over data potentially indefinitely), without adding a new feature. The ticket however is still tagged as a 'new feature' and 'minor', so I wanted to ping you because I think my patch solves their root issue, doesn't add a new feature, and has a large potential impact on memory usage in core-async heavy programs.

😻 1
alexmiller 2020-12-03T19:08:24.296400Z

maybe a new ticket would be better?

✔️ 1
2020-12-03T20:12:01.296900Z

https://clojure.atlassian.net/browse/ASYNC-234 is the new ticket, I'll move the patch over, and see if I can get some graphs of differences in memory use pre/post patch

alexmiller 2020-12-03T20:59:00.297400Z

thanks, that is a good writeup