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

Staging -> prod #6856

Merged
merged 2 commits into from
Feb 11, 2025
Merged

Staging -> prod #6856

merged 2 commits into from
Feb 11, 2025

Conversation

BacLuc
Copy link
Contributor

@BacLuc BacLuc commented Feb 11, 2025

@BacLuc and @pmattmann would like to deploy just these 2 commits from staging to prod.

The reason is:
it seems that these 2 commits improve the response times for the user dramatically.

Before:
https://gist.github.com/BacLuc/3f1dcd8bb374f483e71615a8d65d4d0a

after:
https://gist.github.com/BacLuc/ba1a65e67e970b75897ec284f63e4ae4

with 20 threads
https://gist.github.com/BacLuc/dfe77a165c9b296d10318febb762f826
(did not help, all threads were idling, and the cpu usage of the DB was 100%)

with 8 threads
https://gist.github.com/BacLuc/b447179f85c475101b7220b23eb7dfdf

BacLuc and others added 2 commits February 11, 2025 20:42
May be can then handle more of the simple queries,
while the long queries block a smaller portion
of the overall thread count.
Where-Clause on a ToMany-Association
@BacLuc BacLuc requested a review from a team February 11, 2025 20:55
@BacLuc BacLuc temporarily deployed to staging February 11, 2025 21:10 — with GitHub Actions Inactive
@carlobeltrame carlobeltrame temporarily deployed to staging February 11, 2025 21:11 Destroyed
@pmattmann
Copy link
Member

/fast-forward/

@pmattmann
Copy link
Member

/fast-forward

@BacLuc BacLuc merged commit 7a6fc2c into prod Feb 11, 2025
154 checks passed
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