OHDSI / GIS

https://ohdsi.github.io/GIS
Apache License 2.0
9 stars 9 forks source link

Data types should be changed in gaiadb #268

Open kzollove opened 11 months ago

kzollove commented 11 months ago

These data types are holdovers from the prototype of the data model. Some of them (varchar(x) in particular) go against some best practices for postgres. In general, they should be more simple unless they provide some benefit of being complex.

Data model can be viewed here

Consider:

This shouldnt be a hard problem. This will be a win if the data types used are more uniform.

kzollove commented 11 months ago

@tibbben would love your input on this

p-talapova commented 8 months ago

Hi @kzollove While these considerations may not be new, they are still relevant. In the context of OMOP convention with PostgreSQL: