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

Performance improvements #34

Closed

jgallagher wants to merge 3 commits into lib:master from jgallagher:perf

Contributor

jgallagher commented Jul 29, 2012

Two performance improvements that address issue #29:

Wrap the underlying net.Conn in a bufio.Reader (biggest improvement)
Use the Postgres "Simple Query" interface for Exec()s with no parameters (this helps some too, and it's what psyocopg2 is doing under the hood)

jgallagher added 3 commits

July 29, 2012 00:56


          Use bufio.Reader

a6c22b1


          Use Postgres simple query for Execer with 0 args.

a8d26bb


          Use simpleQuery for prepared Exec()s, too.

459be90

Contributor

bmizerany commented Jul 31, 2012

Merged into my master.

bmizerany closed this

bgentry mentioned this pull request

TLS broken with 'unexpected EOF' from #34 / a6c22b11 #38

Closed

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