Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[BUG] save_to/load_from should be dispatched from coordinator, rather than client #1982

Closed
sighingnow opened this issue Aug 23, 2022 · 0 comments
Closed

[BUG] save_to/load_from should be dispatched from coordinator, rather than client #1982

sighingnow opened this issue Aug 23, 2022 · 0 comments
Assignees
@zhanglei1949
Labels
component:coordinator enhancement New feature or request good first issue Good for newcomers

Comments

@sighingnow
Copy link
Collaborator

Describe the bug

It is not so good to require kubectl on the client side.

Additional context
Add any other context about the problem here.
@sighingnow sighingnow added the good first issue Good for newcomers label Jun 6, 2023
@zhanglei1949 zhanglei1949 self-assigned this Jun 14, 2023
@zhanglei1949 zhanglei1949 mentioned this issue Jun 21, 2023
Merged
zhanglei1949 added a commit that referenced this issue Jun 21, 2023
@zhanglei1949 @sighingnow
[good-first-issue]Move save_to/load_from from client side to coordi…
f0a779e 
…nator (#2917)

[good-first-issue] Current graph serialization and deserialization is performed at client
side, which should be done on coordiantor.
related issue #1982

---------

Co-authored-by: Tao He <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zhanglei1949 zhanglei1949

Labels
component:coordinator enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sighingnow @zhanglei1949