apache / apache/datafusion

Refactor `TableFunctionImpl` to a separate module

Offen
#13,613 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement
Vorherrschende Sprache
Rust
Sterne
9.3k
Forks
2.4k
Ø Merge
3 T. 11 Std.
Gemergte PRs (30 T.)
360

Beschreibung

### Is your feature request related to a problem or challenge?

Background: https://github.com/apache/datafusion/pull/13540#discussion_r1857679120
After https://github.com/apache/datafusion/pull/13540 is merged, there is a refactor can be done:
- Move `TableFunctionImpl` to a separate module
- Add `name()` method under the trait to keep it consistent with other functions (Like user-defined aggregate function)

### Describe the solution you'd like

_No response_

### Describe alternatives you've considered

_No response_

### Additional context

_No response_

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginnen Sie mit der Diskussion unter https://github.com/apache/datafusion/pull/13540#discussion_r1857679120 und suchen Sie TableFunctionImpl im aktuellen DataFusion-Quellcode. Als erledigt gilt die Aufgabe, wenn die Implementierung in einem separaten Modul erfolgt und das Trait eine name()-Methode bereitstellt, die mit den anderen Function-Traits konsistent ist; im Issue werden keine auszuführenden Tests genannt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

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

Neue Issues direkt in Ihr Postfach

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