apache / apache/datafusion

RFC: Rename OptimizerConfig to OptimizerProps

Offen
#6,741 5 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement
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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.