[jira] [Commented] (ACCUMULO-3925) Remove unnecessary warning messages from FATE ops

["Josh Elser (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ACCUMULO-3925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14605780#comment-14605780 ] 

Josh Elser commented on ACCUMULO-3925:
--------------------------------------

One interesting point about making this change is that we would need to only thrown a known-ignorable exception from FATE op implementations because this log message is coming from the FATE framework itself.

If we just wrap and consume the exceptions, it will change the semantics of the FATE op itself which is a little scary.

> Remove unnecessary warning messages from FATE ops
> -------------------------------------------------
>
>                 Key: ACCUMULO-3925
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3925
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: fate
>            Reporter: Josh Elser
>            Assignee: Josh Elser
>            Priority: Minor
>             Fix For: 1.8.0
>
>
> {noformat}
> Failed to execute Repo, tid=76367ba00e847e10
>     ThriftTableOperationException(tableId:null, tableName:ctt_000, op:CREATE, type:EXISTS, description:null)
>         at org.apache.accumulo.master.tableOps.Utils.checkTableDoesNotExist(Utils.java:54)
>         at org.apache.accumulo.master.tableOps.PopulateZookeeper.call(PopulateZookeeper.java:54)
>         at org.apache.accumulo.master.tableOps.PopulateZookeeper.call(PopulateZookeeper.java:30)
>         at org.apache.accumulo.master.tableOps.TraceRepo.call(TraceRepo.java:57)
>         at org.apache.accumulo.fate.Fate$TransactionRunner.run(Fate.java:72)
>         at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
>         at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
>         at org.apache.accumulo.fate.util.LoggingRunnable.run(LoggingRunnable.java:35)
>         at java.lang.Thread.run(Thread.java:745) 
> {noformat}
> {noformat}
> Cannot move tables to a new namespace by renaming. The namespace for nspc_000.ctt_002 does not match ctt_001 
> {noformat}
> Lots of WARN messages seen during randomwalk that aren't system-level warning messages. They are perfectly normal and expected. We should not be putting a warning to the monitor for this messages. The exception thrown back to the client is sufficient.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)