mre/hyperjson

Speed up boolean encoding/decoding

Aberta

#68 aberto em 21 de set. de 2019

Ver no GitHub
 (1 comentário) (0 reação) (0 responsável)Python (41 forks)github user discovery
enhancementgood first issuehacktoberfesthelp wanted

Métricas do repositório

Stars
 (511 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

From our benchmarks we can see that we are consistently slower than everyone else when serializing/deserializing boolean values. We should fix that.

orjson is using an unsafe block to create a reference to a boolean: https://github.com/ijl/orjson/blob/03d55e99a953ce93cedc05f03e4b63b0bcbbcc7a/src/decode.rs#L81-L96

This avoids additional allocations. For comparison, this is our code at the moment:

https://github.com/mre/hyperjson/blob/ded13b4100638aa32fe19dc477f5cfe3e704893c/src/lib.rs#L475-L480

I wonder if we could achieve comparable performance without using unsafe. @konstin, any idea? Maybe there was a recent development in pyo3 that we could leverage here?

Guia do colaborador

Direção de pesquisa
Investigue as diferenças de benchmark entre hyperjson e orjson para serialização booleana. Examine a abordagem unsafe do orjson e explore alternativas seguras usando PyO3 ou a biblioteca padrão do Rust para evitar alocações.
Pilha de tecnologia
pythonrust
Domain
backend
Tipo Issue
Performance
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PythonRust
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50