Remove snowflake to feast type mapping for arrays (not supported)

JohnLemmonMedely · JohnLemmonMedely · commit 7cc1cc02411e · 2024-01-08T14:56:00.000-06:00
diff --git a/sdk/python/feast/type_map.py b/sdk/python/feast/type_map.py
@@ -651,7 +651,6 @@ def snowflake_type_to_feast_value_type(snowflake_type: str) -> ValueType:
         "TIMESTAMP_TZ": ValueType.UNIX_TIMESTAMP,
         "TIMESTAMP_LTZ": ValueType.UNIX_TIMESTAMP,
         "TIMESTAMP_NTZ": ValueType.UNIX_TIMESTAMP,
-        "ARRAY": ValueType.STRING,  # Actual list type cannot be inferred, so failing to a String instead
     }
     return type_map[snowflake_type]
 
diff --git a/sdk/python/tests/integration/materialization/test_snowflake.py b/sdk/python/tests/integration/materialization/test_snowflake.py
@@ -146,7 +146,6 @@ def test_snowflake_materialization_consistency_internal_with_lists(
         now = datetime.utcnow()
 
         full_feature_names = True
-        # TODO use both tz-naive & tz-aware timestamps to test that they're both correctly handled
         start_date = (now - timedelta(hours=5)).replace(tzinfo=utc)
         end_date = split_dt
         fs.materialize(

Original file line number	Diff line number	Diff line change
`@@ -651,7 +651,6 @@ def snowflake_type_to_feast_value_type(snowflake_type: str) -> ValueType:`
`651`	`651`	`"TIMESTAMP_TZ": ValueType.UNIX_TIMESTAMP,`
`652`	`652`	`"TIMESTAMP_LTZ": ValueType.UNIX_TIMESTAMP,`
`653`	`653`	`"TIMESTAMP_NTZ": ValueType.UNIX_TIMESTAMP,`
`654`		`- "ARRAY": ValueType.STRING, # Actual list type cannot be inferred, so failing to a String instead`
`655`	`654`	`}`
`656`	`655`	`return type_map[snowflake_type]`
`657`	`656`