Refactor `TableFunctionImpl` to a separate module
- 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
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