Binding a Decimal infinity to a compiled statement raises OverflowError
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
- 84/100
Línea de trabajo
Empieza en drivine/cypher/sql/parameters.py, en _jsonable (línea 110), donde, según el issue, se convierten los valores Decimal. Compara cómo se manejan Decimal("NaN") y el infinito de float; después, sigue el recorrido del parámetro de la sentencia compilada hasta esta función. Se considera terminado cuando los infinitos Decimal positivo y negativo se enlazan como infinitos de float sin provocar OverflowError; el issue no indica ningún archivo de prueba, así que localiza o añade cobertura para este recorrido.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
A parameter bound as Decimal("Infinity") or Decimal("-Infinity") to a statement compiled to SQL raises OverflowError: cannot convert Infinity to integer from drivine/cypher/sql/parameters.py:110 (_jsonable). This happens for any statement, before the compiler reads what the parameter is for.
The review of #120 found it with round(1234.5, $p) and p = Decimal("-Infinity"). It predates that change.
A float infinity is read as one. A Decimal infinity should be too: read it as a float infinity, as Decimal("NaN") is read as NaN, rather than converted to an integer.
drivine4j and drivine have no Decimal parameter type, so no port is needed.
- Lenguaje dominante
- Python
- Estrellas
- 1
- Forks
- 0
- Merge medio
- 1 h 16 min
- PR fusionados (30 d)
- 74
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 liberation-data/drivine-python
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
liberation-data/drivine-python#152 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
liberation-data/drivine-python#150 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
liberation-data/drivine-python#138 ·
Los mantenedores suelen responder en 1 día
-
Cypher on SQL stores: all(), any(), none() and single() are two-valued where the predicate is nullAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
liberation-data/drivine-python#159 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
liberation-data/drivine-python#158 ·
Los mantenedores suelen responder en 1 día
Todos los issues de liberation-data/drivine-python
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
521xueweihan/HelloGitHub#4115 ·
-
get_lldp_neighbors_detail: query asks for 'max-address' (typo for 'mac-address'), so remote_chassis_id is always emptyPosiblemente ocupada @Jah-yee la tomó hoy. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
-
lesson-submission pending-review
Dificultad 1/5 Menos de una hora Aptitud para principiantes 74/100
Ikalus1988/MisakaNet#3899 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
aws-samples/sample-per-model-bedrock#549 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100