fix(data-warehouse): Dont need to error on binary types anymore (Post…

…Hog#29052)
sloppyjuicy · Feb 21, 2025 · 14a93cf · 14a93cf
1 parent 5ced32e
commit 14a93cf
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/posthog/temporal/data_imports/pipelines/pipeline/hogql_schema.py b/posthog/temporal/data_imports/pipelines/pipeline/hogql_schema.py
@@ -27,6 +27,9 @@ def add_field(self, field: pa.Field, column: pa.ChunkedArray) -> None:
         if existing_type is not None and existing_type != StringDatabaseField.__name__:
             return
 
+        if pa.types.is_binary(field.type):
+            return
+
         hogql_type: type[DatabaseField] = DatabaseField
 
         if pa.types.is_time(field.type):
@@ -43,8 +46,6 @@ def add_field(self, field: pa.Field, column: pa.ChunkedArray) -> None:
             hogql_type = BooleanDatabaseField
         elif pa.types.is_integer(field.type):
             hogql_type = IntegerDatabaseField
-        elif pa.types.is_binary(field.type):
-            raise Exception("Type 'binary' is not a supported column type")
         elif pa.types.is_string(field.type):
             hogql_type = StringDatabaseField