apache / apache/datafusion

RFC: Rename OptimizerConfig to OptimizerProps

Aperta
#6,741 5 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?

Following up PR #6706, I think the name `OptimizerConfig` no longer make sense because it contains per-query execution state such as query execution start time and alias generator.

### Describe the solution you'd like

I think `OptimizerConfig` trait should be renamed to `OptimizerContext` and the existing `OptimizerContext` which is a concrete class should be renamed to `OptimizerState`.

### Describe alternatives you've considered

Leave it as it is

### Additional context

cc @alamb @jackwener

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia esaminando PR #6706 e individuando le definizioni e i riferimenti di OptimizerConfig e OptimizerContext. Il titolo dell’issue e le rinominazioni richieste non concordano, quindi conferma i nomi previsti prima di modificare i riferimenti; il lavoro è completato quando i nomi concordati vengono applicati in modo coerente.

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

Valutazione

Stack tecnologico
rust
Ambito
databases
Tipo di issue
Refactoring
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
20/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.