Schema data type parsing needs hardening
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 72/100
Línea de trabajo
Comienza localizando el parser del tipo de datos del esquema y compara la sintaxis que acepta con el patrón regex de iceberg-go referenciado. Ajusta el parsing para que se rechacen las entradas malformadas con paréntesis de cierre adicionales, mientras se siguen aceptando los tipos válidos; después, añade o ejecuta pruebas del parser que cubran estos casos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Apache Iceberg Rust version
main
Describe the bug
Today, the data type parsing can parse some really weird combinations such as decimal(5, 8)))))))))))))))))))))))))))))))))))))))))))))))))))))))))))))))).
This is not a big issue, but I think it would make more sense for us to parse a stricter subset - not overly strict, but one that is predictable.
To Reproduce
No response
Expected behavior
We should reject types that contain additional ) or otherwise appears malformed.
I'd recommend we adopt a regex pattern, similar to iceberg-go. https://github.com/apache/iceberg-go/blob/8832cf696b5cc2a54767b5f2fa4c06b138c04495/types.go#L35-L40
Willingness to contribute
I can contribute a fix for this bug independently
- Lenguaje dominante
- Rust
- Estrellas
- 1.4k
- Forks
- 574
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 66
Preparar el entorno
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de apache/iceberg-rust
-
datafusion
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
apache/iceberg-rust#3297 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
apache/iceberg-rust#3285 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
apache/iceberg-rust#3280 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
apache/iceberg-rust#3234 · 2 reacciones ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
apache/iceberg-rust#3222 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de apache/iceberg-rust
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
trezor/trezor-firmware#7997 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
smol-machines/smolvm#1489 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día