crate / crate/crate-python

Clarify JSON marshalling of `Decimal` values

Abierto
#751 7 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Python
Estrellas
85
Forks
34
Merge medio
3 d 16 h
PR fusionados (30 d)
3

Descripción

## About

How to marshal values of Python's [Decimal](https://docs.python.org/3/library/decimal.html) type.

## Status quo

Currently, the library is conveying Decimal values as strings.

https://github.com/crate/crate-python/blob/7c7a88521d48a9d5225dbd3842f3c4c07fafd4d8/src/crate/client/http.py#L105-L106

## See also

Others apparently need serialization to float.

```python
if isinstance(obj, Decimal):
return float(obj)
```
-- https://github.com/crate-workbench/meltano-target-cratedb/blob/148a2d5ec7131658c1a74aed7d91f23befc81341/target_cratedb/sqlalchemy/patch.py#L65-L66

## References

- https://github.com/crate-workbench/meltano-target-cratedb/issues/52

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Lee src/crate/client/http.py en las líneas 105-106 y, después, compara el tratamiento de Decimal mostrado en la referencia enlazada de meltano-target-cratedb y en el issue #52. Resuelve qué representación JSON se requiere para los valores Decimal y documenta o prueba el comportamiento acordado; el issue está terminado cuando esa representación está definida explícitamente y verificada de forma coherente.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
python
Área
api, databases
Tipo de issue
Nueva funcionalidad
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.