-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Open
Description
Context
PR #5827 added dbt integration with TTL (time-to-live) for feature views.
Problem
The default TTL is hardcoded to 1 day, which is very aggressive and may not be suitable for most production use cases. Many feature pipelines run weekly or have longer retention requirements.
Current Behavior
# In dbt_import.py:64-68
@click.option(
"--ttl-days",
type=int,
default=1, # Very short!
show_default=True,
help="TTL (time-to-live) in days for feature views",
)Proposed Changes
- Increase default to 7 days - More reasonable for production use cases
- Make default configurable - Allow users to set default in config file
- Add validation - Warn if TTL is unusually short (< 1 day) or long (> 365 days)
Example
default=7, # More reasonable defaultOr support config override:
# feature_store.yaml
dbt:
default_ttl_days: 30Migration Note
This is a breaking change for users who relied on the 1-day default. Consider:
- Documenting the change in release notes
- Providing migration guide
- Adding deprecation warning in interim release
Related
Metadata
Metadata
Assignees
Labels
No labels