Skip to content

Transition heatmap granule encoding to binary#56

Closed
erlewa wants to merge 3 commits intomainfrom
erl/feat/bincode
Closed

Transition heatmap granule encoding to binary#56
erlewa wants to merge 3 commits intomainfrom
erl/feat/bincode

Conversation

@erlewa
Copy link
Contributor

@erlewa erlewa commented May 5, 2025

Heatmap Granule data is currently being sent to the client as a json string. By switching this encoding to binary using bincode we can dramatically reduce the size of transferred data.

  • Updated heatmap-client/src/ingest/request.rs
    • Swapped serde_json::from_str to bincode::decode_from_slice
    • Tidied up code for requesting heatmap granule data
  • Updated heatmap-servcie/src/query.rs
    • Swapped Serde Serialize to use Bincode Encode. Serde Serialize and Deserialize traits are still implemented as the redis cache relies on them, something that I will look into fixing if I have time

@erlewa erlewa self-assigned this May 5, 2025
@erlewa erlewa closed this May 7, 2025
@erlewa
Copy link
Contributor Author

erlewa commented May 7, 2025

This PR was consolidated into #58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant