Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incremental fixes for lock contention #945

Merged
merged 9 commits into from
Sep 6, 2023

Conversation

rkachowski
Copy link

@rkachowski rkachowski commented Aug 24, 2023

Description

Upstream patches for addressing lock contention, with the exclusion of removing foreign key constraints on tables across the db schema. This was removed due to concerns with reversibility raised during the last data-services review meeting.

Other changes

  • Skipping non deterministic tests

Tested

  • Run on rc1staging
  • Passes unit tests

Issues

@github-actions
Copy link

github-actions bot commented Aug 24, 2023

Unit Test Results

       6 files  ±0     366 suites  ±0   3m 50s ⏱️ +3s
2 641 tests  - 1  2 567 ✔️  - 1  74 💤 ±0  0 ±0 
2 655 runs   - 1  2 579 ✔️  - 1  76 💤 ±0  0 ±0 

Results for commit 1dd0425. ± Comparison against base commit 92fbd0e.

♻️ This comment has been updated with latest results.

@rkachowski rkachowski force-pushed the dhutch/drop_address_foreign_keys branch from e49b304 to defd25e Compare September 5, 2023 13:50
@rkachowski rkachowski force-pushed the dhutch/drop_address_foreign_keys branch from defd25e to 1dd0425 Compare September 6, 2023 10:27
@rkachowski rkachowski marked this pull request as ready for review September 6, 2023 10:33
@rkachowski rkachowski requested a review from a team as a code owner September 6, 2023 10:33
@rkachowski rkachowski changed the title Drop addresses foreign key constraints across schema Incremental fixes for lock contention Sep 6, 2023
@rkachowski rkachowski merged commit 6f5a4e9 into master Sep 6, 2023
27 checks passed
@rkachowski rkachowski deleted the dhutch/drop_address_foreign_keys branch September 6, 2023 12:10
rkachowski added a commit that referenced this pull request Sep 7, 2023
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.

3 participants