apache / apache/datafusion-sqlparser-rs
Remove or document second field in `Value::Number`
- 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