[
https://issues.apache.org/jira/browse/THRIFT-917?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ed Ceaser updated THRIFT-917:
-----------------------------
Attachment: THRIFT-917-HandleRuntimeExceptions.patch
OK so I realize now that the java server tests are mostly designed to be
integration level tests of each server... testing the various failure cases
like this one is a bigger project than fixing this bug.
So anyways I think its reasonable to take this patch now, and then add unit
tests in order to test failure behavior at a later date.
> THsHaServer should not accept an ExecutorService without catching
> RejectedExecutionException
> --------------------------------------------------------------------------------------------
>
> Key: THRIFT-917
> URL: https://issues.apache.org/jira/browse/THRIFT-917
> Project: Thrift
> Issue Type: Bug
> Components: Java - Library
> Affects Versions: 0.5
> Reporter: Eric Jensen
> Assignee: Ed Ceaser
> Attachments: THRIFT-917-HandleRuntimeExceptions.patch
>
>
> https://issues.apache.org/jira/browse/THRIFT-250 introduced the ability to
> pass in an executor, but depending on the queue type your executor uses it
> can throw an unchecked RejectedExecutionException which it looks to me would
> kill the selector thread. we need to handle the general error behavior of an
> ExecutorService if we accept one as a parameter
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
