[jira] [Updated] (ACCUMULO-2696) Master has misleading log message in case of failure for status thread

["Sean Busbey (JIRA)" <ji...@apache.org>]

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

Sean Busbey updated ACCUMULO-2696:
----------------------------------

    Description: 
the status thread is a worker within the Master that handles internal state transitions. It contains this handler in a catch-all block to ensure it keeps working:

{noformat}
        } catch (Throwable t) {
          log.error("Error balancing tablets", t);
          UtilWaitThread.sleep(WAIT_BETWEEN_ERRORS);
        }
{noformat}

while balancing does happen in this thread (in a call to updateStatus before this catch), the try block encompasses a great deal of additional code. so this message can be misleading about where to look for problems. It should be updated to state that there was a problem iwth transitioning states and should probably indicate both that it will retry and how long it will pause before doing so.

  was:
the status thread is a worker within the Master that handles internal state transitions. It contains this handler in a catch-all block to ensure it keeps working:

{noformat}
        } catch (Throwable t) {
          log.error("Error balancing tablets", t);
          UtilWaitThread.sleep(WAIT_BETWEEN_ERRORS);
        }
{noformat}

No balancing happens in this thread, so this message is misleading about where to look for problems. It should be updated to state that there was a problem iwth transitioning states and should probably indicate both that it will retry and how long it will pause before doing so.


> Master has misleading log message in case of failure for status thread
> ----------------------------------------------------------------------
>
>                 Key: ACCUMULO-2696
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2696
>             Project: Accumulo
>          Issue Type: Bug
>          Components: master
>    Affects Versions: 1.4.0, 1.5.0, 1.6.0
>            Reporter: Sean Busbey
>            Priority: Minor
>              Labels: logging, newbie
>             Fix For: 1.6.1, 1.7.0
>
>
> the status thread is a worker within the Master that handles internal state transitions. It contains this handler in a catch-all block to ensure it keeps working:
> {noformat}
>         } catch (Throwable t) {
>           log.error("Error balancing tablets", t);
>           UtilWaitThread.sleep(WAIT_BETWEEN_ERRORS);
>         }
> {noformat}
> while balancing does happen in this thread (in a call to updateStatus before this catch), the try block encompasses a great deal of additional code. so this message can be misleading about where to look for problems. It should be updated to state that there was a problem iwth transitioning states and should probably indicate both that it will retry and how long it will pause before doing so.



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