RFC: Rename OptimizerConfig to OptimizerProps
- 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