Skip to content

refactor(concurrent-control): make locks redis-first#1673

Merged
earayu merged 2 commits into
mainfrom
weihong/phase8-redis-first-concurrent-control
Apr 25, 2026
Merged

refactor(concurrent-control): make locks redis-first#1673
earayu merged 2 commits into
mainfrom
weihong/phase8-redis-first-concurrent-control

Conversation

@earayu

@earayu earayu commented Apr 25, 2026

Copy link
Copy Markdown
Collaborator

Scope

Task #38: concurrent_control Redis-first hardening.

Gates

  • uv run --extra test python -m pytest tests/unit_test/concurrent_control tests/unit_test/test_modularization_boundaries.py -q -> 110 passed, 3 warnings
  • uv run ruff format --check aperag/concurrent_control tests/unit_test/concurrent_control -> pass
  • uv run ruff check aperag/concurrent_control tests/unit_test/concurrent_control -> pass
  • git diff --check -> pass
  • rg "get_or_create_lock\\(" aperag -g '*.py' -g '!aperag/concurrent_control/manager.py' -> no production callers
  • rg "lock_type=\\\"threading\\\"|\\\"threading\\\"\\)" aperag -g '*.py' -g '!aperag/concurrent_control/manager.py' -> no production callers
  • rg "evaluation_service|from aperag\\.concurrent_control\\.redis_lock import RedisLock|RedisLock\\(" aperag -g '*.py' -> only concurrent_control internals

@earayu earayu merged commit 92d019f into main Apr 25, 2026
0 of 2 checks passed
@earayu earayu deleted the weihong/phase8-redis-first-concurrent-control branch April 25, 2026 06:05
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