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

Native postgresql driver #2

Closed
torhve opened this issue Apr 25, 2013 · 2 comments
Closed

Native postgresql driver #2

torhve opened this issue Apr 25, 2013 · 2 comments

Comments

@torhve
Copy link
Contributor

torhve commented Apr 25, 2013

There has recently been released a native lua cosocket driver for postgres here:
https://github.com/azurewang/lua-resty-postgres

I did some simple benchmarking ("SELECT 1") and the native driver came out roughly three times as fast for this simple test. I have not done thorough benchmarking.

But maybe the biggest favorable is getting error messages back to the programmer.

Would you consider adding support for this driver?
What about other drivers, lua-resty-mysql, etc, easily doable to abstract out the postgresl-specific parts ?

@leafo
Copy link
Owner

leafo commented Apr 25, 2013

Awesome, I was aware of this but hadn't had a chance to benchmark it yet. I'll look into implementing it shortly. Being able to have error messages would be very helpful.

I'm not against adding other drivers (specifically mysql). It's not a high priority for me personally because Postgres is what I use. If other people show some interest then I don't mind adding support.

@torhve
Copy link
Contributor Author

torhve commented Apr 25, 2013

Alright. If for no other reason, the TechEmpower benchmark uses MySQL. And I'd love to see lapis included in that benchmark.

@torhve torhve closed this as completed Aug 8, 2014
leafo pushed a commit that referenced this issue Sep 14, 2015
Merge pull request #331 from DigitalVeer/master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants