apache / apache/datafusion-sqlparser-rs

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

Ouverte
#363 0 commentaires 2 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
Rust
Étoiles
3.5k
Forks
772
Merge moyen
4 j 9 h
PR mergées (30 j)
17

Description

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.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Commencez par localiser la définition de Value::Number ainsi que chaque constructeur et pattern match qui l’utilise. Vérifiez les dialectes SQL pris en charge afin de déterminer si le champ long est requis ; le travail sera terminé lorsque la variante sera auto-documentée ou que le champ aura été supprimé, que toutes les références compileront et que les tests existants passeront.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
rust, sql
Domaine
databases
Type d'issue
Refactorisation
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.