-
Notifications
You must be signed in to change notification settings - Fork 381
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
Support for PostgreSQL #1298
Comments
Referencing #1298, this PR adds the SQL creation script for a postgres data store and a stripped down reset script. I've done my best to translate the MySQL <> Postgres types; however they may not be a perfect mapping and it's possible that there'll need to be further changes made but I figured it's a good starting point for now.
What's the status of this work? @vishalkuo Is the postgres Trillian storage used anywhere? |
Apologies - I have no had time to look into this at all. It looks like @bigjimnolan has also started some work on this. It might be worth following up on his PRs. |
I see only one commit from @bigjimnolan here. Jim, could you comment whether this is actually being used / tested / developed? |
Could you also clarify whether you use the Just to let you know, the PR #1919 disables postgres storage provider in |
Got it. I believe I do use the provided binaries. Apologies for the delay on this and I appreciate the patience/follow up |
Closing this for now. Feel free to reopen if you actively work on PostgreSQL support. |
Boo! Am I the only one waiting for postgres support? In my experience maria doesn't quite hack it under load. Would you guys merge a PR if it magically appeared? |
Hi, It's not that we're opposed to Postgres per se. Our concerns are around the costs of ongoing maintenance, support and CI / build infrastructure, which will multiply by the number of supported storage types. We did look at Postgres when we started the project but could not find an equivalent to Galera in terms of replication characteristics and the tests with what was available at the time were not very encouraging. However, it should be possible to use a Postgres storage implementation hosted outside of our repository. It will require the |
Hi @Martin2112, Trillian newbie here. A Postgres compatible and scalable database out there is CockroachDB / CockroachCloud. |
Yes we looked at that when the project started but it wasn't stable enough to be used in production. Maybe it's moved on now but the issue of us having limited resources for maintenance still exists. |
We do not have the resources to maintain this storage backend (#1298).
PostgreSQL support has been added to Trillian in #3644. |
Following a conversation with @RJPercival on the trillian slack channel, I think it'd be helpful to add support for a postgres backed data store. I'd like to use this issue for further discussion/tracking progress.
I'll likely borrow heavily from https://github.com/Martin2112/trillian/tree/pgsql/storage/pgsql as a starting point
The text was updated successfully, but these errors were encountered: