◐ Shell
clean mode source ↗

fix: Added unix_timestamp_val in _serialize_val by ntkathole · Pull Request #5659 · feast-dev/feast

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@ntkathole

Copy link Copy Markdown

Member

What this PR does / why we need it:

The fix now allows entities with ValueType.UNIX_TIMESTAMP to be successfully serialized and written to Redis (and other online stores) without errors.

Which issue(s) this PR fixes:

Fixes #5634

Signed-off-by: ntkathole <nikhilkathole2683@gmail.com>

@ntkathole ntkathole self-assigned this

Oct 12, 2025

@ntkathole ntkathole requested a review from a team as a code owner

October 12, 2025 14:46

@franciscojavierarceo franciscojavierarceo merged commit 35a8423 into feast-dev:master

Oct 13, 2025

21 checks passed

franciscojavierarceo pushed a commit that referenced this pull request

Oct 14, 2025
# [0.55.0](v0.54.0...v0.55.0) (2025-10-14)

### Bug Fixes

* Added unix_timestamp_val in _serialize_val ([#5659](#5659)) ([35a8423](35a8423))
* BatchFeatureView transformation should persist in Registry Ser/Deserialization ([3364bad](3364bad))
* Check if DynamoDB table exists before create ([#5658](#5658)) ([e7fd506](e7fd506))
* Fix the link to Expedia in the Go Feature Server readme. ([3ed0163](3ed0163))

### Features

* Add Claude instructions ([#5651](#5651)) ([4807a52](4807a52))
* Groups and Namespaces based authorization, for Users and Service Accounts ([#5619](#5619)) ([da6257c](da6257c))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@franciscojavierarceo franciscojavierarceo franciscojavierarceo approved these changes

Assignees

@ntkathole ntkathole

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Writing to redis fails when one of the entities is a timestamp.

2 participants

@ntkathole @franciscojavierarceo