Sub-issue of #477 (defect 1 only).
Problem
cms_params() in accumulator_factory.rs only reads row_num/col_num, but the planner (sketch.rs) emits depth/width for CountMinSketch. Every plain-CMS aggregation throws:
WARN ... CMS config missing required parameter: row_num
and never ingests. Regression from #397, which made params required without adding the depth/width alias on this path.
cms_heap_params() (for CountMinSketchWithHeap) already accepts depth/width first — the fix is to mirror that pattern in cms_params().
Fix
In cms_params(), try depth/width before row_num/col_num (matching cms_heap_params()).
Sub-issue of #477 (defect 1 only).
Problem
cms_params()inaccumulator_factory.rsonly readsrow_num/col_num, but the planner (sketch.rs) emitsdepth/widthforCountMinSketch. Every plain-CMS aggregation throws:and never ingests. Regression from #397, which made params required without adding the
depth/widthalias on this path.cms_heap_params()(forCountMinSketchWithHeap) already acceptsdepth/widthfirst — the fix is to mirror that pattern incms_params().Fix
In
cms_params(), trydepth/widthbeforerow_num/col_num(matchingcms_heap_params()).