Skip to content
This repository has been archived by the owner on Aug 2, 2023. It is now read-only.

refactor: Migrate distributed to common #570

Merged
merged 3 commits into from
Apr 1, 2022

Conversation

achimnol
Copy link
Member

@achimnol achimnol commented Apr 1, 2022

Make PgAdvisoryLock a manager-local implementation detail.

@achimnol achimnol added this to the 22.03 milestone Apr 1, 2022
@achimnol achimnol self-assigned this Apr 1, 2022
@codecov
Copy link

codecov bot commented Apr 1, 2022

Codecov Report

Merging #570 (486b50c) into main (a32c812) will decrease coverage by 0.12%.
The diff coverage is 76.47%.

@@            Coverage Diff             @@
##             main     #570      +/-   ##
==========================================
- Coverage   49.35%   49.23%   -0.13%     
==========================================
  Files          55       55              
  Lines        9137     9109      -28     
==========================================
- Hits         4510     4485      -25     
+ Misses       4627     4624       -3     
Impacted Files Coverage Δ
src/ai/backend/manager/pglock.py 69.23% <69.23%> (ø)
src/ai/backend/manager/idle.py 30.63% <100.00%> (+0.24%) ⬆️
src/ai/backend/manager/scheduler/dispatcher.py 26.91% <100.00%> (+0.18%) ⬆️
src/ai/backend/manager/server.py 60.36% <0.00%> (-1.58%) ⬇️
src/ai/backend/manager/api/auth.py 48.44% <0.00%> (+0.51%) ⬆️
src/ai/backend/manager/models/utils.py 80.85% <0.00%> (+2.83%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a32c812...486b50c. Read the comment docs.

@achimnol achimnol merged commit 4d74bd7 into main Apr 1, 2022
@achimnol achimnol deleted the refactor/migrate-distributed-to-common branch April 1, 2022 13:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant