-
Notifications
You must be signed in to change notification settings - Fork 18k
x/perf/storage: support postgres for db #49121
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
Comments
I'm not sure whether it's in scope of CC @prattmic. |
I am slightly surprised to see someone running another production version of this. :) What is your use case? You simply want a custom/internal instance of perf.golang.org for your projects? I am not particularly opposed to postgres support, but I also worry about maintenance; I suspect that it would break and we wouldn't even notice. FWIW, we'll probably be changing these packages up a fair bit as part of #48803. |
The use case is to track benchmark data for projects that are not open sourced and where we don't want to expose the content of benchmark results. I was hoping to just start up a local instance of perf.golang.org. It looks like the goal of #48803 is pretty much exactly what I'm trying to accomplish. Is that going to replace perf.golang.org or be an additional tool? |
Current plan (we think) is that perf.golang.org will hold the benchmark data; for your purposes, sounds like we want to make that something that can be (re)configured w/o too much trouble. |
Cleaning open tabs, noticed this, do you have a change-for-postgres already written, and available to look at? I do worry that we might break it, but if it's not large, maybe? I ask because I've messed with postgres in the past, had reason to use it recently in another context. |
I have a change but I haven't had time to fully test or complete it. chriscasola@0fc0a1c |
Oh dear. Thanks for that work, that makes me grumpy at the sql driver authors, some expect It might be interesting to get rid of gratuitous incompatibilities, do the BIGINTs really need to be nonstandardly UNSIGNED? |
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (
go env
)?go env
OutputWhat did you do?
I'm trying to use the
x/perf/storage
app in an environment that requires postgresql instead of mysql.What did you expect to see?
I'd expect postgres to work properly.
What did you see instead?
The create table statements and some of the queries are not compatible with postgres.
I'd be happy to contribute this change if you are open to supporting postgres in addition to the already supported mysql and sqlite.
The text was updated successfully, but these errors were encountered: