Skip to content

feat: Add UUID and TIME_UUID as feature types (#5885) #5951

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.

Sign up for GitHub

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

Merged
ntkathole merged 15 commits into from
Apr 1, 2026
Merged

feat: Add UUID and TIME_UUID as feature types (#5885) #5951

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
6991f44
feat: Add UUID and TIME_UUID as feature types (#5885)
soooojinlee Feb 8, 2026
55065f2
test: Add unit tests for UUID type support
soooojinlee Feb 8, 2026
53c2513
style: Fix ruff lint and formatting issues
soooojinlee Feb 8, 2026
678dc72
feat: Add dedicated UUID/TIME_UUID proto fields to Value.proto
soooojinlee Feb 8, 2026
8e15939
fix: Address review feedback for UUID type support
soooojinlee Feb 8, 2026
a098c3c
fix: Address review feedback for UUID type support
soooojinlee Feb 8, 2026
6c9ba63
fix: Address review feedback
soooojinlee Feb 8, 2026
ac7a61d
fix: Convert uuid.UUID to string for Arrow and JSON serialization
soooojinlee Feb 12, 2026
fe0bfb2
feat: Add UUID_SET/TIME_UUID_SET support and update type system docs
soooojinlee Feb 17, 2026
732088a
fix: Preserve PDF_BYTES/IMAGE_BYTES enum values and add missing SET t…
soooojinlee Feb 17, 2026
5d637c5
fix: Correct misleading comment in Set.__init__
soooojinlee Feb 17, 2026
e4b0a86
refactor: Extract UUID Arrow conversion into helper and move import t…
soooojinlee Feb 18, 2026
b9b15d2
fix: Handle UUID types in _proto_value_to_transport_value for JSON se…
soooojinlee Mar 7, 2026
3c07e12
chore: Regenerate protobuf files with UUID type support
soooojinlee Mar 27, 2026
7b600cb
fix: Fix mypy type ignore comments for UUID collection conversions
soooojinlee Mar 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: Handle UUID types in _proto_value_to_transport_value for JSON se… 
…rialization

Return UUID proto fields as plain strings instead of falling through to
feast_value_type_to_python_type which converts them to uuid.UUID objects
that are not JSON-serializable, causing TypeError during HTTP transport.

Signed-off-by: soojin <soojin@dable.io>
  • Loading branch information
@soooojinlee @ntkathole
soooojinlee authored and ntkathole committed Apr 1, 2026
commit b9b15d206b72408289576223c3a6edbd67760452
10 changes: 10 additions & 0 deletions sdk/python/feast/infra/online_stores/remote.py
View file
Open in desktop
Comment thread
devin-ai-integration[bot] marked this conversation as resolved.
Original file line number Diff line number Diff line change
Expand Up @@ -115,6 +115,16 @@ def _proto_value_to_transport_value(proto_value: ValueProto) -> Any:
if val_attr in ("map_list_val", "struct_list_val"):
return [json.dumps(v) for v in feast_value_type_to_python_type(proto_value)]

# UUID types are stored as strings in proto — return them directly
# to avoid feast_value_type_to_python_type converting to uuid.UUID
# objects which are not JSON-serializable.
if val_attr in ("uuid_val", "time_uuid_val"):
return getattr(proto_value, val_attr)
if val_attr in ("uuid_list_val", "time_uuid_list_val"):
return list(getattr(proto_value, val_attr).val)
if val_attr in ("uuid_set_val", "time_uuid_set_val"):
return list(getattr(proto_value, val_attr).val)

return feast_value_type_to_python_type(proto_value)

def online_write_batch(
Expand Down