apache/pinot

Unify some behavior in TableDataManager

Geschlossen

#9.752 geöffnet am 07.11.2022

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)Java (1.234 Forks)batch import
bugcleanupgood first issue

Repository-Metriken

Stars
 (4.937 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

In the TableDataManager:

  • void addSegment(File indexDir, IndexLoadingConfig indexLoadingConfig) is never used in production code (the only non-testing caller addOfflineSegment() is not wired up)
  • void addSegment(String segmentName, TableConfig tableConfig, IndexLoadingConfig indexLoadingConfig) can be simplified because TableConfig is included in the IndexLoadingConfig
  • void addOrReplaceSegment(String segmentName, IndexLoadingConfig indexLoadingConfig, SegmentZKMetadata zkMetadata, @Nullable SegmentMetadata localMetadata) should be broken into 2 methods: addSegment() and refreshSegment()
  • Loading committed segment into real-time table should share the same logic as loading segment into offline table. Currently the logic is diverged, which causes moving tier not happening for real-time table

Ideally we want to keep the following methods:

  • addSegment
  • addConsumingSegment (real-time only)
  • refreshSegment
  • reloadSegment
  • removeSegment

Contributor Guide

Research-Richtung
Analysieren Sie die aktuellen TableDataManager Methoden: identifizieren Sie den ungenutzten addSegment, vereinfachen Sie addSegment mit redundanter TableConfig, teilen Sie addOrReplaceSegment in addSegment und refreshSegment auf und gleichen Sie die Logik zum Laden von committeten Segmenten in Echtzeit und Offline Tabellen an.
Tech Stack
java
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitJavaApache Pinot
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60