4paradigm / 4paradigm/OpenMLDB

duplicate `Put` code for inserting aggr_table

Offen
#1,658 0 Kommentare 0 Reaktionen 2 zugewiesene Personen Beansprucht von @nautaa Auf GitHub ansehen
enhancement storage-engine
Vorherrschende Sprache
C++
Sterne
1.7k
Forks
331
Ø Merge
12 T. 12 Std.
Gemergte PRs (30 T.)
1

Beschreibung

**Describe the feature you'd like**

Now `aggregator` uses the `Table::Put` API to put the data, and manually calls the `replicator` APIs to write to the log. Duplicate logics with the `Tablet::Put`.

We have to extract a common function to achieve the same objective.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Look at the aggregator's use of Table::Put and replicator APIs, and compare with Tablet::Put in the codebase. Identify common logic for inserting into aggr_table and writing to the log. Focus on extracting a shared function to eliminate duplication. Check existing tests for Put operations to ensure refactoring doesn't break functionality.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
backend, databases
Issue-Typ
Refactoring
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.