crate / crate/crate-python

Clarify JSON marshalling of `Decimal` values

Aberta
#751 7 comentários 0 reações 0 responsáveis Ver no GitHub
Linguagem predominante
Python
Estrelas
85
Forks
34
Merge médio
3d 1h
PRs com merge (30d)
4

Descrição

## 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

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Leia src/crate/client/http.py nas linhas 105-106 e, em seguida, compare o tratamento de Decimal mostrado na referência vinculada de meltano-target-cratedb e no issue #52. Resolva qual representação JSON é necessária para valores Decimal e documente ou teste o comportamento acordado; o issue estará concluído quando essa representação estiver explícita e for verificada de forma consistente.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
python
Domínio
api, databases
Tipo de issue
Funcionalidade
Dificuldade
5/5
Tempo estimado
Mais de uma semana
Status de atividade
Estagnada
Clareza
Precisa de esclarecimento
Facilidade para iniciantes
35/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.