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.

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

bufio: optimize bufio.Reader.WriteTo() and bufio.Writer.ReadFrom() #6373

Open
gopherbot opened this issue Sep 12, 2013 · 12 comments

Comments

Projects
None yet
4 participants
@gopherbot
Copy link

commented Sep 12, 2013

by valyala:

Current implementations of bufio.Reader.WriteTo() and bufio.Writer.ReadFrom() can be
optimized further for common cases such as copying big files to / from network
connections.

See the following topic for details -
https://groups.google.com/d/msg/golang-nuts/oYYggbLntVE/DMsqEyIU-yIJ
@robpike

This comment has been minimized.

Copy link
Contributor

commented Sep 13, 2013

Comment 1:

Labels changed: added priority-later, performance, removed priority-triage.

Status changed to Accepted.

@rsc

This comment has been minimized.

Copy link
Contributor

commented Nov 27, 2013

Comment 2:

Labels changed: added go1.3maybe.

@rsc

This comment has been minimized.

Copy link
Contributor

commented Dec 4, 2013

Comment 3:

Labels changed: added release-none, removed go1.3maybe.

@rsc

This comment has been minimized.

Copy link
Contributor

commented Dec 4, 2013

Comment 4:

Labels changed: added repo-main.

@gopherbot

This comment has been minimized.

Copy link
Author

commented Jan 16, 2014

Comment 5 by pongad:

Working on it!
@gopherbot

This comment has been minimized.

Copy link
Author

commented Jan 23, 2014

Comment 6 by pongad:

https://golang.org/cl/53560043/
@gopherbot

This comment has been minimized.

Copy link
Author

commented Feb 21, 2014

Comment 7 by valyala:

Is there any progress on this issue?
@bradfitz

This comment has been minimized.

Copy link
Member

commented Feb 27, 2014

Comment 8:

This issue was updated by revision b00e477.

LGTM=iant, gri
R=golang-codereviews, iant, gri
CC=golang-codereviews
https://golang.org/cl/69220046
@gopherbot

This comment has been minimized.

Copy link
Author

commented Jun 17, 2014

Comment 9 by pongad:

I would like to continue working on this. Unless someone would like to work on issue
7744 and tackle this in one go?
@bradfitz

This comment has been minimized.

Copy link
Member

commented Jun 17, 2014

Comment 10:

Go ahead and do this. Now's the time.
I am skeptical on issue #7744.
@gopherbot

This comment has been minimized.

Copy link
Author

commented Jul 1, 2014

Comment 11 by pongad:

https://golang.org/cl/103710044/
The benchmark does not look encouraging. Please see my first comment on the CL for more
detail.
@gopherbot

This comment has been minimized.

Copy link
Author

commented Jul 1, 2014

Comment 12:

CL https://golang.org/cl/103710044 mentions this issue.

@rsc rsc added this to the Unplanned milestone Apr 10, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.