From 1f787dc4a4b9ff421b96470be99ba692d5071cab Mon Sep 17 00:00:00 2001 From: Matthieu Patou Date: Sat, 2 Mar 2024 06:39:43 +0000 Subject: [PATCH] Force _event_time to be a datetime type Summary It makes things so much easier in applications like superset because the column is properly used. Test plan Changed the type, created a new wheel and used it in superset Reviewers: haneeshr --- src/rockset_sqlalchemy/sqlalchemy/dialect.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/rockset_sqlalchemy/sqlalchemy/dialect.py b/src/rockset_sqlalchemy/sqlalchemy/dialect.py index db5ccf4..f6c1810 100644 --- a/src/rockset_sqlalchemy/sqlalchemy/dialect.py +++ b/src/rockset_sqlalchemy/sqlalchemy/dialect.py @@ -96,6 +96,8 @@ def _get_table_columns(self, connection, table_name, schema): field_type, fields[0], schema, table_name ) ) + if field[0] == "_event_time": + field_type = "timestamp" columns.append( { "name": field[0],