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

Support for PostgreSQL #1298

Closed
vishalkuo opened this issue Sep 19, 2018 · 11 comments
Closed

Support for PostgreSQL #1298

vishalkuo opened this issue Sep 19, 2018 · 11 comments
Assignees
Labels

Comments

@vishalkuo
Copy link
Contributor

vishalkuo commented Sep 19, 2018

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

AlCutter pushed a commit that referenced this issue Sep 24, 2018
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.
@pav-kv
Copy link
Contributor

pav-kv commented Oct 21, 2019

What's the status of this work?

@vishalkuo Is the postgres Trillian storage used anywhere?

@vishalkuo
Copy link
Contributor Author

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.

@pav-kv
Copy link
Contributor

pav-kv commented Oct 22, 2019

I see only one commit from @bigjimnolan here. Jim, could you comment whether this is actually being used / tested / developed?

@pav-kv
Copy link
Contributor

pav-kv commented Oct 22, 2019

Could you also clarify whether you use the trillian_log_server and trillian_log_signer binaries that we provide in this repository, or you have your own main.go files?

Just to let you know, the PR #1919 disables postgres storage provider in main.go files of trillian_log_server and trillian_log_signer by default given the development status, until we know that it needs to be enabled. Feel free to submit the corresponding "enabling" PR, or ask me/team to do that.

@vishalkuo
Copy link
Contributor Author

Got it. I believe I do use the provided binaries. Apologies for the delay on this and I appreciate the patience/follow up

@pav-kv
Copy link
Contributor

pav-kv commented Jan 21, 2021

Closing this for now. Feel free to reopen if you actively work on PostgreSQL support.

@pav-kv pav-kv closed this as completed Jan 21, 2021
@marcopeereboom
Copy link

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?

@Martin2112
Copy link
Contributor

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 main,go files for the binaries to be forked and a storage provider to be written to wrap up the flags but that should be more or less it. If there are obstacles to doing this then we'd probably be willing to look at making smaller changes to support it.

@aPaleBlueDot
Copy link

Hi @Martin2112, Trillian newbie here. A Postgres compatible and scalable database out there is CockroachDB / CockroachCloud.

@Martin2112
Copy link
Contributor

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.

pphaneuf added a commit that referenced this issue Mar 30, 2021
We do not have the resources to maintain this storage backend (#1298).
@robstradling
Copy link
Contributor

PostgreSQL support has been added to Trillian in #3644.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants