mre/hyperjson

Speed up boolean encoding/decoding

Aperta

#68 aperta il 21 set 2019

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Python (41 fork)github user discovery
enhancementgood first issuehacktoberfesthelp wanted

Metriche repository

Star
 (511 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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?

Guida contributor

Direzione di ricerca
Esamina le differenze di benchmark tra hyperjson e orjson per la serializzazione booleana. Analizza l'approccio unsafe di orjson ed esplora alternative sicure usando PyO3 o la libreria standard di Rust per evitare allocazioni.
Tech stack
pythonrust
Dominio
backend
Tipo issue
Performance
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonRust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50