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

[IM] WriteClient Lifecycle Management #10366

Closed
yunhanw-google opened this issue Oct 8, 2021 · 4 comments
Closed

[IM] WriteClient Lifecycle Management #10366

yunhanw-google opened this issue Oct 8, 2021 · 4 comments
Assignees
@erjiaqing
Labels
Interaction Model Work V1.0

Comments

@yunhanw-google
Copy link
Contributor

Problem

Like the work being done for CommandSender, need to update the lifecycle management of WriteClient to not be allocated by the stack, but rather, allocated and managed by the application.
@yunhanw-google yunhanw-google changed the title [IM] WriteClient Lifecycle Management: [IM] WriteClient Lifecycle Management Oct 8, 2021
@mrjerryjohns
Copy link
Contributor

Since we already have a WriteClientHandle, I'm inclined to say that for v1, that suffices. Going to close this issue for now.

@mrjerryjohns mrjerryjohns reopened this Nov 19, 2021
@mrjerryjohns
Copy link
Contributor

Sorry - I meant that it will be punted out of TE7.5

@yufengwangca
Copy link
Contributor

@erjiaqing I think this task has already been completed, no?

@yunhanw-google
Copy link
Contributor Author

#13248

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@erjiaqing erjiaqing

Labels
Interaction Model Work V1.0
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@andy31415 @erjiaqing @mrjerryjohns @yunhanw-google @yufengwangca @woody-apple