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

Queued request timeout metrics #735

Closed
wants to merge 4 commits into from
Closed

Conversation

apaliy
Copy link
Contributor

@apaliy apaliy commented May 28, 2020

Metrics added for Queued request timeout

@apaliy apaliy requested review from DGarbar and rpanchyk May 28, 2020 13:30
@apaliy apaliy self-assigned this May 28, 2020
@rpanchyk rpanchyk requested review from schernysh and removed request for DGarbar May 28, 2020 14:59
@rpanchyk
Copy link
Contributor

rpanchyk commented Jun 3, 2020

This PBS-Go ported PR covers infrastructure issue, when request is waiting on Load Balancer and doesn't require to be processed since timeout exceeded - https://github.com/prebid/prebid-server/blob/b5993cd0e5912638b03cea039466673325dee40b/config/config.go#L67

The request_time_in_queue and request_timeout_in_queue headers looks like specific for particular infrastructure.

PBS-Java need TBD. Closing.

@rpanchyk rpanchyk closed this Jun 3, 2020
@rpanchyk rpanchyk deleted the queued-request-timeout-metrics branch June 3, 2020 12:04
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.

2 participants