apache / apache/datafusion

Refactor `TableFunctionImpl` to a separate module

Aperta
#13,613 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub
enhancement
Lingua principale
Rust
Stelle
9.3k
Fork
2.4k
Merge medio
3g 11h
PR unite (30g)
360

Descrizione

### 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_

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia dalla discussione all’indirizzo https://github.com/apache/datafusion/pull/13540#discussion_r1857679120 e individua TableFunctionImpl nel codice sorgente corrente di DataFusion. Il lavoro è completato quando l’implementazione si trova in un modulo separato e il trait espone un metodo name() coerente con gli altri trait delle funzioni; l’issue non indica alcun test da eseguire.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
rust
Ambito
databases
Tipo di issue
Refactoring
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.