-
Notifications
You must be signed in to change notification settings - Fork 0
Labels
P0Critical priorityCritical priorityanalysis-modeAnalysis Mode featureAnalysis Mode featurebugSomething isn't workingSomething isn't workingin-progressWork currently in progressWork currently in progress
Description
問題
PR #108 (GroupByStrategyResolver) 和 #109 (IAnalysisCache) 各自新增了 AnalysisQueryEngine 的無參數建構子。GitHub auto-merge 未偵測到衝突(不同行),但合併後產生 CS0111 — 重複建構子定義。
影響
dotnet8 分支目前無法編譯 — 這是 P0 阻塞問題。
修復方案
將兩組建構子合併為一組,同時接受 GroupByStrategyResolver 和 IAnalysisCache?:
public AnalysisQueryEngine() : this(GroupByStrategyResolver.Default, null) { }
public AnalysisQueryEngine(GroupByStrategyResolver resolver, IAnalysisCache? cache = null)
{
_resolver = resolver ?? throw new ArgumentNullException(nameof(resolver));
_cache = cache;
}同時清除殘留的 ExecuteGroupBy 和 BuildGroupKey 私有方法(已被提取到 InProcessGroupByStrategy)。
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
P0Critical priorityCritical priorityanalysis-modeAnalysis Mode featureAnalysis Mode featurebugSomething isn't workingSomething isn't workingin-progressWork currently in progressWork currently in progress