Experimental IOLocalContextStorage

[rossabaker]

I think this is doomed, but putting it out there to continue the discussion in #202.

Edit by @NthPortal: it is not necessarily doomed

[iRevive]

Does it seamlessly propagate the context? At least from the examples, it seems it does.

[rossabaker]

I don't think it works at all, because the dispatched fibers have their own IOLocal state. But typelevel/cats-effect#3636 may open a new path.

[rossabaker]

6b542aa probably belongs on main for general cleanliness. The motivator here is that the Java SDK loads its own testkit if it's present, in preference to any other SPI implementation, like ours.

[rossabaker]

This is going to want to live in an SPI module. Having more than one on the classpath is like multiple slf4j bindings.

[NthPortal]

I just realised there's a small problem of this not being the same type of context used in the API of the java backend, and I'm not entirely sure how to deal with that

[NthPortal]

so, with the new context implementation, if we use our own context and just have the custom ContextStorage unbox it, that should work fine. but we also need to expose access to a Local[F, Context] so that scala API users can access the context

[NthPortal]

just want to drop a note that I've been working on this (with Ross' blessing), and have a working implementation with tests. there's just the teeny, tiny problem that it depends not just on an unreleased cats effect version but on one that's not even merged yet. so, uh. not sure quite what to do about that

[NthPortal]

this is ready for review. I'm reluctant to mark it as not a draft however, until typelevel/cats-effect#3636 is merged

[razvanz]

@iRevive Has this been accidentally closed?

[iRevive]

@razvanz thanks for pointing out! I guess I linked the wrong task/PR, and it closed automatically.

[NthPortal]

thanks for updating this @iRevive!