4paradigm / 4paradigm/OpenMLDB
merge aggregators in `src/storage/aggregator.xx` and `hybridse/src/vm/aggregator.h`
- Lingua principale
- C++
- Stelle
- 1.7k
- Fork
- 331
- Merge medio
- 12g 12h
- PR unite (30g)
- 1
Descrizione
**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.
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- cpp
- Ambito
- backend
- Tipo di issue
- Refactoring
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 45/100