apache / apache/datafusion-sqlparser-rs
Remove or document second field in `Value::Number`
- Lenguaje dominante
- Rust
- Estrellas
- 3.5k
- Forks
- 772
- Merge medio
- 4 d 9 h
- PR fusionados (30 d)
- 17
Descripción
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.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Línea de trabajo
Comienza localizando la definición de Value::Number y cada constructor y pattern match que la utilice. Comprueba los dialectos SQL compatibles para determinar si el campo long es necesario; se considerará terminado cuando la variante sea autodocumentada o se haya eliminado el campo, todas las referencias compilen y las pruebas existentes pasen.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- rust, sql
- Área
- databases
- Tipo de issue
- Refactorización
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 45/100