Skip to content

feat: Add skip_feature_view_validation parameter to push() for ODFVs with missing UDF dependencies #5894

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

Draft
Copilot wants to merge 11 commits into
base: master
Choose a base branch
from
Draft

feat: Add skip_feature_view_validation parameter to push() for ODFVs with missing UDF dependencies #5894

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
ee8a7d0
Initial plan
Copilot Jan 23, 2026
6b2bf7f
Add skip_validation parameter to push method and related methods
Copilot Jan 23, 2026
f9542e5
Fix caching decorator compatibility with skip_udf parameter
Copilot Jan 23, 2026
79d6744
Apply code formatting with ruff
Copilot Jan 23, 2026
a735bfb
Use explicit identity functions instead of lambda for better type safety
Copilot Jan 23, 2026
2433df6
Only skip UDF validation for ODFVs without write_to_online_store
Copilot Jan 23, 2026
72c471e
Minor wording improvement in test documentation
Copilot Jan 23, 2026
885536e
Rename skip_validation to skip_feature_view_validation for API consis…
Copilot Jan 24, 2026
d4dd5f5
Merge branch 'master' into copilot/fix-module-not-found-error
franciscojavierarceo Jan 25, 2026
98f92a2
Merge branch 'master' into copilot/fix-module-not-found-error
franciscojavierarceo Jan 26, 2026
c111fba
Merge branch 'master' into copilot/fix-module-not-found-error
franciscojavierarceo Jan 29, 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
Minor wording improvement in test documentation 
Co-authored-by: franciscojavierarceo <4163062+franciscojavierarceo@users.noreply.github.com>
  • Loading branch information
@franciscojavierarceo
Copilot and franciscojavierarceo committed Jan 23, 2026
commit 72c471eb84ad83690f7d91133e209b679a6533d0
4 changes: 2 additions & 2 deletions sdk/python/tests/unit/test_skip_validation.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -203,8 +203,8 @@ def test_skip_validation_use_case_documentation():

With skip_validation=True:
- push() calls list_all_feature_views(skip_validation=True)
- ODFVs WITHOUT write_to_online_store=True are loaded with dummy UDFs (identity functions)
- ODFVs WITH write_to_online_store=True are loaded normally (UDF is deserialized)
- ODFVs with write_to_online_store=False are loaded with dummy UDFs (identity functions)
- ODFVs with write_to_online_store=True are loaded normally (UDF is deserialized)
- No deserialization of the actual UDF happens for ODFVs that won't execute transformations
- push() can proceed successfully

Expand Down