irmin icon indicating copy to clipboard operation
irmin copied to clipboard

Abstract away from Lwt / Add support for Async

Open hhugo opened this issue 5 years ago • 4 comments

The core of irmin currently depend on Lwt. Could we lift that constraint ?

hhugo avatar Feb 20 '20 15:02 hhugo

I'm not against abstract IO in principle, but I believe we have had issues with trying to unify Lwt and Async semantics before. @dinosaure knows the details there.

It would also be a big refactor at this point, so it would be nice to have some specific use-cases in mind before beginning such an undertaking.

craigfe avatar Feb 20 '20 16:02 craigfe

The use case is simply to experiment with irmin inside an existing async codebase.

hhugo avatar Feb 20 '20 17:02 hhugo

I think it might be better to wrap the Lwt scheduler inside Async for this experiment (see https://discuss.ocaml.org/t/best-practice-on-mixing-async-and-lwt/4473). Do you want to give that a try for the experiment? (ping us on Slack to unblock if stuck)

avsm avatar Feb 20 '20 17:02 avsm

Could be solved by #1325

Ngoguey42 avatar Apr 07 '21 15:04 Ngoguey42

The issue no longer relevant for me

hhugo avatar Jan 09 '24 14:01 hhugo