-
Notifications
You must be signed in to change notification settings - Fork 13
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
Improved stress tests, added automated stress tests #175
Improved stress tests, added automated stress tests #175
Conversation
bf7802a
to
8932426
Compare
e20880b
to
bdc0409
Compare
73525e8
to
2adabcb
Compare
2adabcb
to
2f8f35f
Compare
Changes Unknown when pulling 2f8f35f on kailuowang:topic/automated-stress into * on iheartradio:master*. |
@@ -49,8 +49,8 @@ trait Dispatcher extends Actor with ActorLogging { | |||
|
|||
context watch processor | |||
|
|||
private val autothrottler = settings.autothrottle.foreach { s ⇒ | |||
context.actorOf(Autothrottler.default(processor, s, metricsCollector), "auto-scaler") | |||
private val _ = settings.autothrottle.foreach { s ⇒ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not just remove this assignment all together?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks more functional? without this the compiler warns on abandoned value.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't have a strong opinion either way.
KanaloaSimulations.scala
for detailsKanaloaLocalSimulation
for detail