apache / apache/datafusion-sqlparser-rs

Remove or document second field in `Value::Number`

Aperta
#363 0 commenti 2 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
Rust
Stelle
3.5k
Fork
772
Merge medio
4g 9h
PR unite (30g)
17

Descrizione

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.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Inizia individuando la definizione di Value::Number e ogni costruttore e pattern match che la utilizza. Controlla i dialetti SQL supportati per determinare se il campo long è necessario; il lavoro è completato quando la variante è autoesplicativa oppure il campo è stato rimosso, tutti i riferimenti compilano e i test esistenti passano.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
rust, sql
Ambito
databases
Tipo di issue
Refactoring
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.