-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
refactoringCode refactoring without changing functionalityCode refactoring without changing functionalitytech-debtTechnical debt that needs addressingTechnical debt that needs addressing
Milestone
Description
Part of SRP Refactoring (Priority 3: Data Processing)
Current Problem
cache_query_generic()
in src/cache/query.c mixes initialization, cache attempt, and fallback logic.
Refactoring Plan
Split into:
query_from_cache()
- Cache query onlyquery_from_journal()
- Journal scan onlyselect_query_strategy()
- Decision logic
Acceptance Criteria
- Query strategies clearly separated
- Strategy selection isolated
- Each path independently testable
- Fallback logic explicit
- Performance monitoring possible
- Clean interfaces
Technical Notes
- Consider strategy pattern
- Add query performance metrics
- Make strategies pluggable
Related to SRP_REFACTORING_PLAN.md Priority 3.2
Metadata
Metadata
Assignees
Labels
refactoringCode refactoring without changing functionalityCode refactoring without changing functionalitytech-debtTechnical debt that needs addressingTechnical debt that needs addressing