ray_beam_runner icon indicating copy to clipboard operation
ray_beam_runner copied to clipboard

Rename immutable context managers to *Context

Open pabloem opened this issue 3 years ago • 4 comments

Specifuically RunnerContextManager and BundleContextManager are what we need to improve.

pabloem avatar Jun 15 '22 22:06 pabloem

Also, make RunnerContextManager fully immutable (will also help to persist it with ray.put) to avoid the cost of passing it around ever and over.

pabloem avatar Jun 15 '22 22:06 pabloem

let me c if i can start looking into this issue. @pabloem can you later give some help here if I take this item?

wilsonwang371 avatar Dec 14 '22 06:12 wilsonwang371

I think it's fine if @flyingImer picks this up ? : )

pabloem avatar Feb 08 '23 17:02 pabloem

@pabloem not sure I follow the RunnerContextManager one - not seeing any reference in the code base, am I missing anything here?

Do you mean RayRunnerExecutionContext and RayBundleContextManager ?

flyingImer avatar Feb 20 '23 20:02 flyingImer