[
https://issues.apache.org/jira/browse/ACCUMULO-4335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15320850#comment-15320850
]
Sean Busbey commented on ACCUMULO-4335:
---------------------------------------
There are several in examples and tests that I'm not worried about. There are a
couple in the server code. They look like they're mostly in early init code
(like when the GC is first checking for a lock, or when Master is doing an
upgrade) or fail-safe (like if we somehow get a bare exception in
TabletServer.main) and they all are non-zero exit.
The only issue I can see is this one in TabletServer
{code}
} catch (Exception ex) {
log.error("Uncaught exception in TabletServer.main, exiting", ex);
System.exit(1);
} finally {
DistributedTrace.disable();
}
{code}
That finally block will never happen if we hit the exception path. But HTrace
itself has a shutdown hook to ensure things are closed off, so probably fine?
Since these are essentially an unrelated problem I'd prefer to defer correcting
them, if you don't mind.
> Error conditions that result in a Halt call should ensure non-zero process
> exit code
> ------------------------------------------------------------------------------------
>
> Key: ACCUMULO-4335
> URL: https://issues.apache.org/jira/browse/ACCUMULO-4335
> Project: Accumulo
> Issue Type: Bug
> Components: gc, tserver
> Affects Versions: 1.6.5, 1.7.0
> Reporter: Sean Busbey
> Assignee: Sean Busbey
> Fix For: 1.8.0, 2.0.0
>
> Attachments: ACCUMULO-4335.1.patch
>
>
> There are several places where we get some fatal error and exit with a 0
> code, indicating no error.
> e.g. when the TabletServer loses the ability to monitor zk.
> (this impacts 1.6.0+, but I can only select 1.6.5 from 1.6.z)
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)