whitfin / cachex

100%

master: 100%

LAST BUILD ON BRANCH fix/nested-transaction-locking

branch: SELECT

CHANGE BRANCH

x

• No branch selected
• arcanemachine-patch
• callers-tweaks
• cmcintosh/two-element-tuple-for-fetch
• fallback-callers
• fix/accessed-limit-cache-miss
• fix/compilation-warnings
• fix/nested-transaction-locking
• issue-382
• issue-386
• issue-418
• main
• master
• minor_edits
• patch-1
• support-elixir-118
• tkowal/406-populate-callers-for-better-test-integrations
• update-default-frequency
• update-fetch-doc
• update_docs

Build # d60cdbd9917f8f93f0e4ded25519adefd65665c2-PR-439

Build Type

Pull #439

github

Committed by makoto-developer

Commit Message

Enforce key locking for nested transactions

Nested transactions previously short-circuited on a boolean flag and ran
the function without acquiring locks, so an inner transaction locking a
different key than the enclosing one would proceed with no locking in place.

Store the set of locked keys in :cachex_transaction instead of a boolean so
a nested transaction can verify that every key it requests is already locked
by the enclosing transaction, raising a Cachex.Error otherwise.

Refs #431

Pull Request Pull Request #439: Enforce key locking for nested transactions (#431)

Coverage Stats

10 of 10 new or added lines in 2 files covered. (100.0%)

811 of 811 relevant lines covered (100.0%)

1081.17 hits per line