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

Transport Hystrix configuration #92

Merged
merged 1 commit into from
Oct 5, 2017

Conversation

mtyni
Copy link
Contributor

@mtyni mtyni commented Sep 27, 2017

Added configurations that can be configured with properties for Hystix. Default global values are:
-Timeout for job is 25 seconds
-Circuit breaker is activated for 20 seconds if 5 requests fail in 100 seconds window.

…ix. Default global values are:

 -Timeout for job is 25 seconds
 -Circuit breaker is activated for 20 seconds if 5 requests fail in 100 seconds window.
@ZakarFin ZakarFin added this to the 1.45.0 milestone Oct 5, 2017
@ZakarFin ZakarFin merged commit 8d8c60b into oskariorg:develop Oct 5, 2017
@ZakarFin
Copy link
Member

ZakarFin commented Oct 5, 2017

This includes new config options for oskari(/transport)-ext.properties for controlling when to stop sending requests to service that doesn't answer fast enough (service might be slowed down by getting too many requests and we don't want to pile up requests that will fail):

# milliseconds as observation window for counting failures before stop sending more WFS-requests to a problematic service (defaults 100 seconds)
oskari.transport.rollingwindow = 100000

# amount of WFS-requests that need to fail/layer in rolling window to do a cooldown (circuit break)
oskari.transport.failrequests = 5

# milliseconds to wait after circuit break until start sending new WFS-requests to problematic WFS-service (defaults 20 seconds)
oskari.transport.sleepwindow = 20000

Also increased the defaut timeout from 15 seconds to 25 - configurable with:

oskari.transport.job.timeoutms = 25000

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