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.

Sign up for GitHub

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

Jump to bottom

Diagnose DB spike issue #4382

Closed
5 of 6 tasks
jcx120 opened this issue May 13, 2022 · 3 comments
Closed
5 of 6 tasks

Diagnose DB spike issue #4382

jcx120 opened this issue May 13, 2022 · 3 comments
Assignees
@paulbellamy @Shaptic @tamirms @2opremio @sreuland @sydneynotthecity @erika-sdf
Labels
horizon

Comments

@jcx120
Copy link

jcx120 commented May 13, 2022 •
edited by sydneynotthecity
Loading

Tasks:

  • Setup Staging environment (full history ingestion etc)
  • Reproduce Issue
    • Setup Staging
    • Replay ALB Logs
  • Analyze stg behavior while its replayed
  • Audit DB queries to verify timeout is happening
@sreuland sreuland mentioned this issue May 19, 2022
Open
sreuland added a commit to sreuland/go that referenced this issue May 24, 2022
@sreuland
stellar#4382: updated the claimable balance SQL query to position LIM…
e3e7dd3 
…IT depending on whether paging cursor parameters are present or not
@paulbellamy
Copy link
Contributor

I suspect #4402 will help with client-side timeout enforcement when the db is unresponsive.

sreuland added a commit to sreuland/go that referenced this issue May 24, 2022
@sreuland
stellar#4382: refined the placement of LIMIT clause on claimant balan…
cc89cc1 
…ce query by claimant id per profiling
sreuland added a commit that referenced this issue May 25, 2022
@sreuland
/db/history: update the claimable balance SQL query (#4398)
da94a37 
* #4382: refined the placement of LIMIT clause on claimant balance query by claimant id per profiling results.
sreuland added a commit that referenced this issue May 26, 2022
@sreuland
/db/history: update the claimable balance SQL query (#4398)
1c42c2b 
* #4382: refined the placement of LIMIT clause on claimant balance query by claimant id per profiling results.
sreuland added a commit that referenced this issue Jun 9, 2022
@sreuland
/db/history: update the claimable balance SQL query (#4398)
7e2220b 
* #4382: refined the placement of LIMIT clause on claimant balance query by claimant id per profiling results.
@sreuland
Copy link
Contributor

@jcx120 , with the incident document and the horizon 2.17.1/2.18.0 releases included optimizations based on findings, this ticket can likely be moved to done?

@jcx120
Copy link
Author

jcx120 commented Jun 10, 2022

Yes - updated to done. thanks

@jcx120 jcx120 closed this as completed Jun 10, 2022
sreuland added a commit to sreuland/go that referenced this issue Aug 7, 2022
@sreuland
stellar#4382: updated the claimable balance SQL query to position LIM…
ed20b8b 
…IT depending on whether paging cursor parameters are present or not
sreuland added a commit to sreuland/go that referenced this issue Aug 7, 2022
@sreuland
stellar#4382: refined the placement of LIMIT clause on claimant balan…
e4d8952 
…ce query by claimant id per profiling
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@paulbellamy paulbellamy

@Shaptic Shaptic

@tamirms tamirms

@2opremio 2opremio

@sreuland sreuland

@sydneynotthecity sydneynotthecity

@erika-sdf erika-sdf

Labels
horizon
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
9 participants
@paulbellamy @Shaptic @tamirms @ire-and-curses @2opremio @sreuland @sydneynotthecity @jcx120 @erika-sdf