[ https://issues.apache.org/jira/browse/ZOOKEEPER-1442?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13262420#comment-13262420 ]
Rakesh R commented on ZOOKEEPER-1442: ------------------------------------- Sorry to jump in late here. Its really a great patch and following are just some thoughts from mine, 1) It would be good moving the literals "logOnly", "logAndExitOnError", "none" to the enum DefaultUncaughtExceptionStrategyType and expose a method getType() or type(), so will avoid hardcoding in multiple places. 2) I agree, presently there is no exception handlers for NettyServer, but I feel would be great provide these strategies to this also. what others feel? > 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