accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Elser (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-3723) TabletServerBatchReaderIterator can eat OOM exceptions
Date Tue, 14 Apr 2015 00:37:12 GMT


Josh Elser commented on ACCUMULO-3723:

Along the same lines, I think we catch Error in at least one place which is a big no-no as
well. IIRC, the Error-catch was related to noticing *when* an OOME happens, but it might be
better removed than any benefit it provides (especially with the {code}-XX:OnOutOfMemoryError="${ACCUMULO_KILL_CMD:-kill
-9 %p}"{code} we have in bin/accumulo now)

> TabletServerBatchReaderIterator can eat OOM exceptions
> ------------------------------------------------------
>                 Key: ACCUMULO-3723
>                 URL:
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.6.0
>            Reporter: John Vines
> Currently in run() it catches Throwable and does no effort to force up catastrophic errors.
In general we shouldn't be catching Throwable unless we know exactly what we're doing.

This message was sent by Atlassian JIRA

View raw message