4paradigm / 4paradigm/OpenMLDB

merge aggregators in `src/storage/aggregator.xx` and `hybridse/src/vm/aggregator.h`

Abierto
#1,533 0 comentarios 0 reacciones 1 asignado Reclamado por @zhanghaohit Ver en GitHub
code-standards
Lenguaje dominante
C++
Estrellas
1.7k
Forks
331
Merge medio
12 d 12 h
PR fusionados (30 d)
1

Descripción

**Describe the feature you'd like**
There are two different places to implement the pre-aggregation logics for the `long window` feature. We should finally merge these in order to avoid code duplication.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

The issue mentions merging aggregators in `src/storage/aggregator.xx` and `hybridse/src/vm/aggregator.h`. First, examine these two files to understand the pre-aggregation logic for the 'long window' feature. Look for overlapping code and design a unified implementation. Check for existing tests related to aggregators to ensure the merge does not break functionality. The goal is to eliminate duplication while maintaining the same behavior.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
cpp
Área
backend
Tipo de issue
Refactorización
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.