mozilla-releng / balrog
90%
main: 90%

Repo Added 08 Jan 2020 02:24PM UTC

Committed 06 Feb 2026 09:23AM UTC coverage: 89.901% (+0.001%) from 89.9%

Committed by

Commit Message

Bug 2013441 - Use orjson to serialize/deserialize objects for the redis cache

The main thing here is that Blobs cannot be deserialized from JSON
directly so we have to provide a specific loader to the redis cache,
which is why we weren't using orjson in the first place. The commit is
mostly plumbing to pass the redis_loads method through all of the right
cache layers/factories.

I originally thought about making this less invasive and just returning
the value as is from the cache and let the caller decide what to do with
what it got back from the cache (for blobs it could either get a dict or
a Blob object based on whether the cache was hit or not, which could
easily be detected by always calling createBlob on the result and
returning early if it was already a Blob). I went back on this because
it felt more fragile and error prone than doing the conversion
explicitly. It's still no perfect still those cache definitions aren't
tested properly and short of deactivating the LRU cache they're hard to
test locally.

I tested this by disabling the LRU cache completely and using an `assert
cache_value == value_getter()` in the cache hit path then hitting
`/update/3/Firefox/42.0/0/Linux_x86_64-gcc3/en-US/release/None/default/default/update.xml`
on the public API. Everything worked fine.

All the other public caches use trivial objects so we don't need special
loaders for them. The admin app doesn't have redis configured.

I removed test_release_blobs_can_log_after_pickling since it's
irrelevant now but decided against reverting
e8cb2ec09 entirely because getting a
logger once per class instead of once per object still sounds like an
improvement we'd want and the other test to deepcopy is also still
relevant for caches using `make_copies`.

2165 of 2546 branches covered (85.04%)

Branch coverage included in aggregate %.

11 of 12 new or added lines in 1 file covered. (91.67%)

5731 of 6237 relevant lines covered (91.89%)

0.92 hits per line

6237 RELEVANT LINES 5731 COVERED LINES

0.92 HITS PER LINE

Builds	Branch	Commit	Type	Ran	Committer	Via	Coverage
#5196	orjson	Bug 2013441 - Use orjson to serialize/deserialize objects for the redis cache The main thing here is that Blobs cannot be deserialized from JSON directly so we have to provide a specific loader to the redis cache, which is why we weren't using or...	Pull #3659	06 Feb 2026 09:32AM UTC	Eijebong	circleci	89.9
#5195	orjson	Bug 2013441 - Use orjson to serialize/deserialize objects for the redis cache The main thing here is that Blobs cannot be deserialized from JSON directly so we have to provide a specific loader to the redis cache, which is why we weren't using or...	Pull #3659	05 Feb 2026 10:39PM UTC	Eijebong	circleci	89.9
#5188	orjson	Update auth0-python to 5.0.0 The changelog is real scary "Major Rewrite" but the authentication API did not get changed and our only usage of the auth0-python library is `from auth0.authentication import Users as auth0_Users`. Now whether it's w...	Pull #3659	05 Feb 2026 01:57PM UTC	Eijebong	circleci	89.9
#5186	orjson	Bug 2013441 - Use orjson to serialize/deserialize objects for the redis cache The main thing here is that Blobs cannot be deserialized from JSON directly so we have to provide a specific loader to the redis cache, which is why we weren't using or...	Pull #3659	05 Feb 2026 01:40PM UTC	Eijebong	circleci	89.9

mozilla-releng / balrog
90%
main: 90%

mozilla-releng / balrog 90% main: 90%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

CHANGE BRANCH x

Relevant lines Covered

Source Files on orjson

Recent builds

mozilla-releng / balrog
90%
main: 90%

README BADGES
x

CHANGE BRANCH
x