← 返回首页
feat: Add lazy initialization and feature service caching (#5924) · Shizoqua/feast@ab9c6fc · 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

Commit ab9c6fc

Browse files
2 peopleauthored and
Shizoqua
committed
feat: Add lazy initialization and feature service caching (feast-dev#5924)
* feat: Add lazy initialization and feature service caching Implement performance optimizations for FeatureStore: - **Lazy Initialization**: Convert registry, provider, and OpenLineage emitter to lazy properties. Reduces cold start from 2.4s to 0.5s (5x improvement). - **Feature Service Caching**: Add caching layer for feature service resolution. Observed 19.6x speedup on cached calls during validation. Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com> * fix: Add type annotations and formatting fixes for performance optimizations * fix: Clear feature service cache on registry refresh When refresh_registry() or plan() methods refresh the registry, the feature service cache must be cleared to avoid serving stale cached results. - Add _clear_feature_service_cache() helper method - Clear cache in refresh_registry() and plan() methods - Prevents data consistency issues with cached feature services * fix: Replace direct _registry access with registry property Fixes AttributeError where tests were directly accessing _registry before lazy initialization. With lazy loading, _registry starts as None and must be accessed through the registry property to trigger initialization. - Replace ._registry. with .registry. in test files - Add runtime error check in registry property for failed initialization - Ensures backward compatibility with existing code patterns Fixes CI test failures in search API tests. * fix: Prevent __repr__ from triggering lazy initialization The __repr__ method was accidentally calling self.registry (property) instead of self._registry (attribute), which would trigger lazy initialization whenever the FeatureStore object was printed or logged. This completely defeated the lazy loading optimization since any debugging, logging, or repr() call would cause full initialization (negating the 2.4s to 0.5s performance gain). - Change self.registry to self._registry in __repr__ method - Preserves lazy loading benefits for debugging/logging scenarios - Maintains correct status reporting without side effects --------- Co-authored-by: Claude Sonnet 4 <noreply@anthropic.com> Signed-off-by: Shizoqua <hr.lanreshittu@gmail.com>
1 parent 91b88b2 commit ab9c6fc

5 files changed

Lines changed: 588 additions & 533 deletions

File tree

0 commit comments

Comments
 (0)

Footer

© 2026 GitHub, Inc.