apache / apache/datafusion-sqlparser-rs
Remove or document second field in `Value::Number`
- Vorherrschende Sprache
- Rust
- Sterne
- 3.5k
- Forks
- 772
- Ø Merge
- 4 T. 9 Std.
- Gemergte PRs (30 T.)
- 17
Beschreibung
Value::Number should be changed from a tuple variant to a struct variant so that the field names are self documenting.
Something like: `Number { value: String, long: bool }`
Possibly the long field could be just removed instead but not im not very familar with various SQL languages.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Beginne damit, die Definition von Value::Number sowie jeden Konstruktor und jedes Pattern-Match zu finden, das sie verwendet. Prüfe die unterstützten SQL-Dialekte, um festzustellen, ob das long-Feld erforderlich ist; fertig ist die Änderung, wenn die Variante selbstdokumentierend ist oder das Feld entfernt wurde, alle Verweise kompilieren und die bestehenden Tests erfolgreich sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- rust, sql
- Bereich
- databases
- Issue-Typ
- Refactoring
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100