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 cockroachdb + yugabytedb as additional engine support #1111

Open
paulm17 opened this issue Mar 2, 2024 · 1 comment
Open

support cockroachdb + yugabytedb as additional engine support #1111

paulm17 opened this issue Mar 2, 2024 · 1 comment
Assignees
Labels
feature request New feature or request

Comments

@paulm17
Copy link

paulm17 commented Mar 2, 2024

Is your feature request related to a problem? Please describe.
This is a feature.

Postgres is a great database, however it's a SPOF. Both cockroachdb and yugabyte are solutions for this issue.

Unfortunately neither support xid8 type.

I do see that SpiceDB (to support cockroachdb) uses a normal timestamp to sidestep this issue.

Describe the solution you'd like

Would be awesome if both cockroachdb and yugabyte are supported to give others options. Looking at the migration files, once one is completed. It's a copy and paste for the other.

Additional context

Would the work entail?

  1. Creating a new folder in the internal / storage directory with migrations tailored for another database.
  2. Changing migration.go to support the other engines.
  3. Changing the database interactivity insert/update/delete to support the other engines?

I suspect it could be a minimal change, due to both databases mimic a postgres connection?

Thanks!

@paulm17 paulm17 added the feature request New feature or request label Mar 2, 2024
@paulm17
Copy link
Author

paulm17 commented May 16, 2024

I had a bit of free time yesterday to finalise this for myself. I now have Yugabyte working with permify.

https://github.com/paulm17/permify

I've added [yugabyte] as a new engine and have found the relevant functions and types (that required minimum 12/13) for version 11. Hopefully this won't cause any unnecessary conflicts or issues.

Eventually Yugabyte will upgrade to 15: yugabyte/yugabyte-db#9797

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

No branches or pull requests

3 participants