[
https://issues.apache.org/jira/browse/ZOOKEEPER-1442?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13262780#comment-13262780
]
Patrick Hunt commented on ZOOKEEPER-1442:
-----------------------------------------
bq. I thought from the email discussion that the default behavior shouldn't
change until the next major release (e.g., 4.0).
I missed that discussion, sorry. As long as we maintain backward compatibility
we can make such a change in a "minor" release. 3.4 to 3.5 say. And in this
case we are maintaining the b/w compat - you can configure the server as you
wish. Additionally there are other cases where the server will exit, and you
have to handle that (see the supervisor section of the admin guide). So really
we're not changing the general behavior of the server here.
@rakesh - there's no need to have the type if you go to a strategy class
approach, really in that case you don't want to have such.
> Uncaught exception handler should exit on a java.lang.Error
> -----------------------------------------------------------
>
> Key: ZOOKEEPER-1442
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1442
> Project: ZooKeeper
> Issue Type: Bug
> Components: java client, server
> Affects Versions: 3.4.3, 3.3.5
> Reporter: Jeremy Stribling
> Assignee: Jeremy Stribling
> Priority: Minor
> Attachments: ZOOKEEPER-1442.patch, ZOOKEEPER-1442.patch
>
>
> The uncaught exception handler registered in NIOServerCnxnFactory and
> ClientCnxn simply logs exceptions and lets the rest of ZooKeeper go on its
> merry way. However, errors such as OutOfMemoryErrors should really crash the
> program, as they represent unrecoverable errors. If the exception that gets
> to the uncaught exception handler is an instanceof a java.lang.Error, ZK
> should exit with an error code (in addition to logging the error).
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira