accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mike Drob (JIRA)" <j...@apache.org>
Subject [jira] [Resolved] (ACCUMULO-2208) Provide better error handling when unable to cleanly close log
Date Tue, 22 Apr 2014 21:40:17 GMT

     [ https://issues.apache.org/jira/browse/ACCUMULO-2208?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Mike Drob resolved ACCUMULO-2208.
---------------------------------

    Resolution: Fixed

> Provide better error handling when unable to cleanly close log
> --------------------------------------------------------------
>
>                 Key: ACCUMULO-2208
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2208
>             Project: Accumulo
>          Issue Type: Bug
>          Components: tserver
>    Affects Versions: 1.5.0
>            Reporter: John Vines
>             Fix For: 1.6.1, 1.7.0, 1.5.1
>
>
> I saw this once over 36 tserver/days, caused by an InvalidArgumentException, so odd occurrence.
It was doing CI but before agitation started so it was a non issue, but it could be. The code
around this is
> {code}for (DfsLogger logger : loggers) {
>         try {
>           logger.close();
>         } catch (DfsLogger.LogClosedException ex) {
>           // ignore
>         } catch (Throwable ex) {
>           log.error("Unable to cleanly close log " + logger.getFileName() + ": " + ex);
>         }
>       }
>       loggers.clear();
>       logSizeEstimate.set(0);{code}
> so it seems like it blindly powered forward. However, because of the frequency of syncs
and such, even if there was agitation the odds of losing data are very minute, but I do believe
their still there.
> But  because the log message just relies on error.toString, I didn't get the violating
stack trace. At the bare minimum, the message should be updated to log the full stack trace.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message