@bascule @yukihiro_matz @headius Please don't. Please. If you want something else, give people a new mechanism. Don't change Thread#[].
@yukihiro_matz @evanphx @headius is overriding thread-local variable methods (e.g. Thread#[]) completely crazy? https://github.com/mperham/sidekiq/issues/318 …
-
-
-
@evanphx@yukihiro_matz@headius I strongly agree! However the problem is legacy code which uses thread locals, e.g. ActiveRecord -
@bascule@yukihiro_matz@headius Is it because Thread#[] respects Fibers? -
@evanphx@yukihiro_matz@headius it's one reason yes. I'd like for thread locals to be "actor locals" inside actors -
@bascule@yukihiro_matz@headius So add Actor#[]. I am sad there is no way to access the non-Fiber aware locals. -
@evanphx@yukihiro_matz@headius how do I deal with e.g. ActiveRecord that already uses Thread#[]? -
@bascule@yukihiro_matz@headius I'd rather you monkey patch AR. -
@evanphx@yukihiro_matz@headius well, I've at least been talking to@tenderlove about fixing AR so it doesn't need thread locals - 4 more replies
New conversation -
Loading seems to be taking a while.
Twitter may be over capacity or experiencing a momentary hiccup. Try again or visit Twitter Status for more information.