Skip to content

feat: Add online sequence configuration foundation - #6848

Open
patelchaitany wants to merge 1 commit into
feast-dev:masterfrom
patelchaitany:feat/rhoaieng-80117-online-config-foundation
Open

patelchaitany wants to merge 1 commit into
feast-dev:masterfrom
patelchaitany:feat/rhoaieng-80117-online-config-foundation

Conversation

@patelchaitany

@patelchaitany patelchaitany commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Add the OnlineConfig dataclass with latest and sequence modes, retention limits, write semantics, and validation.
  • Persist online configuration through FeatureView protobuf serialization and registry apply/update flows.
  • Display online configuration in feature-views describe and document the experimental contract.
  • Add focused coverage for validation, protobuf round trips, backward compatibility, registry apply/update/clear behavior, and CLI output.

Scope

This PR establishes the backward-compatible SDK and registry foundation for future sequence features. Online-store append, sequence retrieval, and backend-specific eviction are intentionally deferred to follow-up PRs.

Acceptance criteria

  • OnlineConfig dataclass with mode, max_length, max_age, and write_mode
  • FeatureView accepts online_config
  • Proto serialization and deserialization
  • Sequence validation requires append writes and positive max_length
  • Existing behavior remains unchanged when online_config is omitted
  • feast apply stores, updates, retrieves, and clears OnlineConfig
  • feature-views describe displays configuration details

Validation

  • 26 focused tests passed
  • Ruff checks passed
  • Scoped mypy checks passed for all changed Python modules
  • Python protobuf generation and local Go protobuf generation validated
  • Pre-commit format, lint, template, and secret checks passed

Full repository mypy currently requires the optional types-PyMySQL stub, which is not installed in the local environment.

Parent epic: RHOAIENG-80117

@codecov-commenter

codecov-commenter commented Sep 18, 2026 •

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 79.10448% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 49.12%. Comparing base (8cfe891) to head (61e8c8f).

Files with missing lines Patch % Lines
sdk/python/feast/online_config.py 76.92% 8 Missing and 4 partials ⚠️
sdk/python/feast/cli/feature_views.py 66.66% 0 Missing and 1 partial ⚠️
sdk/python/feast/infra/registry/registry.py 75.00% 0 Missing and 1 partial ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6848      +/-   ##
==========================================
+ Coverage   49.06%   49.12%   +0.06%     
==========================================
  Files         433      434       +1     
  Lines       54314    54381      +67     
  Branches     7912     7929      +17     
==========================================
+ Hits        26647    26715      +68     
+ Misses      25788    25778      -10     
- Partials     1879     1888       +9     
Flag Coverage Δ
go-feature-server 30.58% <ø> (ø)
python-unit 50.52% <79.10%> (+0.06%) ⬆️
Files with missing lines Coverage Δ
sdk/python/feast/__init__.py 97.22% <100.00%> (+0.07%) ⬆️
sdk/python/feast/batch_feature_view.py 77.46% <100.00%> (+0.32%) ⬆️
sdk/python/feast/feature_view.py 87.26% <100.00%> (+0.24%) ⬆️
sdk/python/feast/cli/feature_views.py 33.08% <66.66%> (+6.02%) ⬆️
sdk/python/feast/infra/registry/registry.py 64.08% <75.00%> (+0.79%) ⬆️
sdk/python/feast/online_config.py 76.92% <76.92%> (ø)

... and 2 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8cfe891...61e8c8f. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@patelchaitany
patelchaitany force-pushed the feat/rhoaieng-80117-online-config-foundation branch 2 times, most recently from 8c9a901 to 899c4ce Compare September 21, 2026 11:29
@patelchaitany
patelchaitany marked this pull request as ready for review September 21, 2026 11:44
@patelchaitany
patelchaitany requested a review from a team as a code owner September 21, 2026 11:44
@patelchaitany
patelchaitany force-pushed the feat/rhoaieng-80117-online-config-foundation branch 4 times, most recently from fdeee98 to 0ef34fd Compare September 28, 2026 08:27
@patelchaitany
patelchaitany force-pushed the feat/rhoaieng-80117-online-config-foundation branch 2 times, most recently from df9bfb2 to a6bc9b0 Compare October 5, 2026 06:07
Signed-off-by: Chaitany Patel <patelchaitany93@gmail.com>
@patelchaitany
patelchaitany force-pushed the feat/rhoaieng-80117-online-config-foundation branch from a6bc9b0 to 61e8c8f Compare October 6, 2026 10:07

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants