-
Notifications
You must be signed in to change notification settings - Fork 4.5k
experimental: add package and move recv buffer pool APIs into it #6692
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
Conversation
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.
Do you have a dependent PR that builds on this by adding/moving things here? Can we include this with that? It doesn't seem very useful to have an experimental package with nothing in it.
Added the recv buffer pool APIs to the experimental package. PTAL. Thanks. |
dialoptions.go
Outdated
return newFuncDialOption(func(o *dialOptions) { | ||
o.recvBufferPool = bufferPool | ||
}) |
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.
return withRecvBufferPool(bufferPool)
experimental/experimental.go
Outdated
* | ||
*/ | ||
|
||
// Package experimental is a placeholder for experimental features that might |
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.
s/placeholder for/ collection of/
RELEASE NOTES: none