-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
feat: convenience factories for Scala Try to StatusReply transform #31783
Conversation
* | ||
* Also note that Akka does not contain pre-build serializers for arbitrary exceptions. | ||
*/ | ||
def fromTryKeepException[T](aTry: Try[T]): StatusReply[T] = new StatusReply(aTry) |
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.
a better name than aTry
?
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.
Just calling it status
could work, same as the Try in the StatusReply itself.
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.
LGTM.
def fromTryKeepException[T](aTry: Try[T]): StatusReply[T] = new StatusReply(aTry) | ||
|
||
/** | ||
* Scala API: Turn a try into a status reply. |
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.
* Scala API: Turn a try into a status reply. | |
* Scala API: Turn a Try into a status reply. |
Co-authored-by: Enno Runne <458526+ennru@users.noreply.github.com>
No description provided.