[GitHub] [incubator-openwhisk] tysonnorris commented on issue #4316: Update version for Akka dependencies and Scala

[GitBox <gi...@apache.org>]

tysonnorris commented on issue #4316: Update version for Akka dependencies and Scala
URL: https://github.com/apache/incubator-openwhisk/pull/4316#issuecomment-469497175
 
 
   @chetanmeh this seems like it will require a code change in `AkkaContainerClient`; I think we should get the same exception handling behavior, regardless of whether the underlying retry mechanism causes the retry, or we exceed the timeout without catching exceptions. I'm not sure what exact behavior changed there, but we may need to add logic for testing the timeout somewhere besides the ```request.map().recoverWith()...``` that is currently there?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services