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

entities."deletedAt" has different type from other timestamp columns #1104

Closed
matthew-white opened this issue Mar 13, 2024 · 0 comments · Fixed by #1145
Closed

entities."deletedAt" has different type from other timestamp columns #1104

matthew-white opened this issue Mar 13, 2024 · 0 comments · Fixed by #1145
Assignees

Comments

@matthew-white
Copy link
Member

In general, our timestamp columns are of type timestamp(3) with time zone, where 3 is the precision value. However, entities."deletedAt" is timestamp without time zone: it differs on "with time zone" and also looks to have a precision value of 6. It looks like it's the only column like that and that other timestamp columns share the same type:

SELECT table_name, column_name, data_type, datetime_precision
FROM information_schema.columns
WHERE data_type ~ 'timestamp' AND table_schema = 'public';

I can't think of an obvious way that these differences would have user-facing effects. For example, we don't filter on entities."deletedAt", so #459 isn't an issue. However, I think it'd be a good idea to make it the same type as other timestamp columns in case the differences lead to subtle issues.

@github-project-automation github-project-automation bot moved this to 🕒 backlog in ODK Central Apr 30, 2024
@ktuite ktuite moved this from 🕒 backlog to ✏️ in progress in ODK Central May 15, 2024
@github-project-automation github-project-automation bot moved this from ✏️ in progress to ✅ done in ODK Central May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ done
Development

Successfully merging a pull request may close this issue.

2 participants