RFC: Rename OptimizerConfig to OptimizerProps
- 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?
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
Beitragsleitfaden
Rechercherichtung
Beginne mit der Prüfung von PR #6706 und finde die Definitionen und Referenzen von OptimizerConfig und OptimizerContext. Der Titel des Issues und die angeforderten Umbenennungen stimmen nicht überein. Bestätige daher die beabsichtigte Benennung, bevor du Referenzen änderst; als erledigt gilt die Aufgabe, wenn die vereinbarten Namen durchgängig verwendet werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- rust
- Bereich
- databases
- Issue-Typ
- Refactoring
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 20/100