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

Use the Postgres UUID column type for UUID values #862

Open
jtwalsh0 opened this issue Sep 4, 2021 · 1 comment
Open

Use the Postgres UUID column type for UUID values #862

jtwalsh0 opened this issue Sep 4, 2021 · 1 comment
Assignees

Comments

@jtwalsh0
Copy link
Member

jtwalsh0 commented Sep 4, 2021

We've been using SQLAlchemy text type for UUID columns rather than Postgres's native UUID column type. (See here for an example.) The latter is 1) more efficient and 2) more robust because it handles inconsistent hyphen use.

The biggest drawback to the Postgres type is that it ties triage to Postgres. Are we hoping to remain database agnostic?

Perhaps I'm being optimistic, but I'd expect this to be an easy change, and I'd be happy to do it if there are no objections.

@nanounanue nanounanue self-assigned this Sep 17, 2021
@nanounanue
Copy link
Contributor

triage is not db agnostic :)

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