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:
[email protected]


With regards,
Apache Git Services

Reply via email to