dssg / triage

General Purpose Risk Modeling and Prediction Toolkit for Policy and Social Good Problems
Other
182 stars 61 forks source link

Use the Postgres UUID column type for UUID values #862

Open jtwalsh0 opened 2 years ago

jtwalsh0 commented 2 years ago

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 commented 2 years ago

triage is not db agnostic :)