-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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.
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
io_uring: update API and docs #38677
Conversation
Signed-off-by: Xie Zhihao <[email protected]>
Signed-off-by: Xie Zhihao <[email protected]>
Signed-off-by: Xie Zhihao <[email protected]>
CC @envoyproxy/api-shepherds: Your approval is needed for changes made to |
/assign @wbpcode |
Signed-off-by: Xie Zhihao <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this updating. And some minor comments are added. Then I can merge this.
api/envoy/extensions/network/socket_interface/v3/default_socket_interface.proto
Outdated
Show resolved
Hide resolved
api/envoy/extensions/network/socket_interface/v3/default_socket_interface.proto
Outdated
Show resolved
Hide resolved
api/envoy/extensions/network/socket_interface/v3/default_socket_interface.proto
Outdated
Show resolved
Hide resolved
…t_interface.proto Co-authored-by: code <[email protected]> Signed-off-by: Xie Zhihao <[email protected]>
…t_interface.proto Co-authored-by: code <[email protected]> Signed-off-by: Xie Zhihao <[email protected]>
Signed-off-by: Xie Zhihao <[email protected]>
Signed-off-by: Xie Zhihao <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks.
Signed-off-by: Xie Zhihao [email protected]
Commit Message: io_uring: update API
Additional Description:
Risk Level: Low
Testing: N/A
Docs Changes: Added
Release Notes: N/A
Platform Specific Features: N/A
Fix #38660