tableau / community-tableau-server-insights

Community-built data sources for answering questions about Tableau Server
MIT License
129 stars 52 forks source link

Force all LUID fields to lowercase #8

Closed mcoles closed 4 years ago

mcoles commented 5 years ago

We should LOWER() all LUID fields in all data sources, because the REST API returns them in all lowercase, and they're stored as uppercase in the PostgreSQL repository. Case sensitivity will prevent a match when data sets from both sources are joined. The whole point of including those fields was to make these data sources easy to use with the REST API, so we can take one pain point out of that process by doing this.