← 返回首页
fix: Added unix_timestamp_val in _serialize_val by ntkathole · Pull Request #5659 · feast-dev/feast · GitHub
Skip to content

Navigation Menu

Toggle navigation
Sign in
Appearance settings
Search or jump to...

Search code, repositories, users, issues, pull requests...

Provide feedback

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

Include my email address so I can be contacted
Cancel Submit feedback

Saved searches

Use saved searches to filter your results more quickly

Cancel Create saved search
Appearance settings
Resetting focus

fix: Added unix_timestamp_val in _serialize_val#5659

Merged
franciscojavierarceo merged 1 commit into
feast-dev:masterfrom
ntkathole:fix_unix_timestamp_val
Oct 13, 2025
Merged

fix: Added unix_timestamp_val in _serialize_val#5659
franciscojavierarceo merged 1 commit into
feast-dev:masterfrom
ntkathole:fix_unix_timestamp_val

Conversation

Copy link
Copy Markdown
Member

ntkathole commented Oct 12, 2025

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

fix: Added unix_timestamp_val in _serialize_val …
ec65fd7
Signed-off-by: ntkathole <nikhilkathole2683@gmail.com>
ntkathole self-assigned this Oct 12, 2025
ntkathole requested a review from a team as a code owner October 12, 2025 14:46
Hide details View details 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
chore(release): release 0.55.0 …
dfb7c50
# [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))
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

franciscojavierarceo franciscojavierarceo approved these changes

Assignees

ntkathole

Labels

ok-to-test

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

Footer

© 2026 GitHub, Inc.