apache / apache/datafusion-sqlparser-rs

[Discussion] Remove use of `dialect_of` macro

Offen
#339 8 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Rust
Sterne
3.5k
Forks
772
Ø Merge
4 T. 9 Std.
Gemergte PRs (30 T.)
17

Beschreibung

What was the purpose of the current `Dialect`?
The `dialect_of` macro makes it difficult to modify your code whenever you add support for Dialect.
And besides the identifier support of Dialect, it seems that it can also support query (Expr).

But... if you have other intentions for macros
Since there is a problem, I would like to discuss it with the maintainers.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Lies die aktuelle Definition von `Dialect` und die Implementierung des Makros `dialect_of`, wobei du dich auf die in der Diskussion erwähnte Unterstützung für Bezeichner und `Expr` konzentrierst. Prüfe die Kommentare der Maintainer, um die vorgesehene Rolle des Makros zu bestimmen. Für den Abschluss sind ein abgestimmtes Design und ein abgestimmter Umfang erforderlich, die im Issue noch nicht festgelegt sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
rust
Bereich
compilers
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.