-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Introduce Buffer::slice API #11001
Introduce Buffer::slice API #11001
Conversation
✅ Deploy Preview for meta-velox canceled.
|
CI failures are not related to this PR, thanks. |
7085bc2
to
1b189d3
Compare
1b189d3
to
1a689a4
Compare
CI failure is not related to this PR, open an issue for it: #11035. |
@mbasmanova , could you help check again? Thanks. |
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.
Overall looks good. Some remaining comments.
01715ab
to
67981b7
Compare
934d079
to
38fa570
Compare
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.
c3e25e7
to
9999345
Compare
@Yuhta has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Conbench analyzed the 1 benchmark run on commit There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
No description provided.