Skip to content

[core] Creating an interface ObjectManager's for GrpcClientManager.#53656

Merged
edoakes merged 2 commits intomasterfrom
irabbani/CORE-1582
Jun 9, 2025
Merged

[core] Creating an interface ObjectManager's for GrpcClientManager.#53656
edoakes merged 2 commits intomasterfrom
irabbani/CORE-1582

Conversation

@israbbani
Copy link
Copy Markdown
Contributor

This is purely refactoring with no new logic:

  • Rename GrpcStubManager to GrpcClientManager
  • Turn GrpcClientManager into an abstract class with an implementation

and cleanup.

Signed-off-by: irabbani <irabbani@anyscale.com>
@israbbani israbbani added the go add ONLY when ready to merge, run all tests label Jun 9, 2025
@israbbani israbbani changed the title [core] Creating an interface ObjectManager's GrpcClientManager. [core] Creating an interface ObjectManager's for GrpcClientManager. Jun 9, 2025
@israbbani israbbani marked this pull request as ready for review June 9, 2025 20:25
@israbbani israbbani requested a review from a team June 9, 2025 20:25
@edoakes edoakes enabled auto-merge (squash) June 9, 2025 20:37
@edoakes edoakes merged commit 2cf7aba into master Jun 9, 2025
6 checks passed
@edoakes edoakes deleted the irabbani/CORE-1582 branch June 9, 2025 21:05
elliot-barn pushed a commit that referenced this pull request Jun 18, 2025
…53656)

This is purely refactoring with no new logic:

- Rename `GrpcStubManager` to `GrpcClientManager`
- Turn `GrpcClientManager` into an abstract class with an implementation

---------

Signed-off-by: irabbani <irabbani@anyscale.com>
Signed-off-by: elliot-barn <elliot.barnwell@anyscale.com>
elliot-barn pushed a commit that referenced this pull request Jul 2, 2025
…53656)

This is purely refactoring with no new logic:

- Rename `GrpcStubManager` to `GrpcClientManager`
- Turn `GrpcClientManager` into an abstract class with an implementation

---------

Signed-off-by: irabbani <irabbani@anyscale.com>
Signed-off-by: elliot-barn <elliot.barnwell@anyscale.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants