René's URL Explorer Experiment

Title: feat: Add UUID and TIME_UUID as feature types (#5885) by soooojinlee · Pull Request #5951 · feast-dev/feast · GitHub

Open Graph Title: feat: Add UUID and TIME_UUID as feature types (#5885) by soooojinlee · Pull Request #5951 · feast-dev/feast

X Title: feat: Add UUID and TIME_UUID as feature types (#5885) by soooojinlee · Pull Request #5951 · feast-dev/feast

Description: What this PR does / why we need it: Adds UUID and TIME_UUID as native Feast feature types, resolving #5885. Currently UUID values must be stored as STRING, which loses type semantics, prevents backend-specific features (e.g. Cassandra timeuuid range queries), and makes PostgreSQL uuid columns infer as STRING. This PR enables users to declare UUID features with Field(name="user_id", dtype=Uuid) and receive uuid.UUID objects from get_online_features().to_dict(). Design Decisions Why two types (UUID vs TIME_UUID)? The issue author explicitly requested distinguishing time-based UUID (uuid1) and random UUID (uuid4). Both serialize identically to string in proto, but separate types allow expressing intent in feature definitions and enable future backend-specific optimizations. Why dedicated proto fields (uuid_val, time_uuid_val)? Following the pattern established by SET types (PR #5888) and UNIX_TIMESTAMP (which reuses int64/Int64List), we add dedicated oneof fields that reuse existing proto scalar types (string and StringList). This allows WhichOneof("val") to identify UUID types directly from the proto message, without requiring a side-channel. Backward compatibility for data stored before this change: OnlineResponse accepts an optional feature_types dict. When data was previously stored as string_val, this metadata enables feast_value_type_to_python_type() to convert it to uuid.UUID. New materializations use uuid_val/time_uuid_val and are identified automatically. Changes Layer Files Description Proto Value.proto, generated *_pb2.py/*_pb2.pyi Add UUID=30, TIME_UUID=31, UUID_LIST=32, TIME_UUID_LIST=33 to ValueType.Enum; add uuid_val, time_uuid_val, uuid_list_val, time_uuid_list_val to Value.oneof Type system value_type.py, types.py Add UUID, TIME_UUID, UUID_LIST, TIME_UUID_LIST enums and Uuid/TimeUuid aliases Type conversion type_map.py Add mappings to ~11 conversion dicts (proto, PyArrow, pandas, PostgreSQL, Couchbase, Snowflake); switch from string_val to uuid_val; add PROTO_VALUE_TO_VALUE_TYPE_MAP entries for UUID fields Online response online_response.py, online_store.py, feature_store.py, utils.py Pass feature_types metadata for backward-compatible deserialization ODFV on_demand_feature_view.py Add UUID/TIME_UUID sample values for schema inference Backward Compatibility Data previously stored as string_val still deserializes correctly via the feature_types side-channel New materializations use dedicated uuid_val/time_uuid_val proto fields feast_value_type_to_python_type(v) without feature_type now returns uuid.UUID for uuid_val fields (previously returned plain string for string_val) PostgreSQL uuid columns now infer as ValueType.UUID (previously ValueType.STRING) Go SDK: proto changes compile without errors; UUID handling logic is not implemented (out of scope) Tests test_types.py: Uuid/TimeUuid ↔ ValueType bidirectional conversion, Array types test_type_map.py: Proto roundtrip with uuid_val, uuid.UUID object return, backward compatibility for string_val, UUID list roundtrip, PostgreSQL mapping All 78 unit tests passing ruff lint and format checks passing

Open Graph Description: What this PR does / why we need it: Adds UUID and TIME_UUID as native Feast feature types, resolving #5885. Currently UUID values must be stored as STRING, which loses type semantics, prevents back...

X Description: What this PR does / why we need it: Adds UUID and TIME_UUID as native Feast feature types, resolving #5885. Currently UUID values must be stored as STRING, which loses type semantics, prevents back...

Opengraph URL: https://github.com/feast-dev/feast/pull/5951

X: @github

direct link

Domain: github.com

Links:

Viewport: width=device-width