[ACCUMULO-3345] Redundant logging on failed multi-scans

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Abandoned
• Component/s: tserver
• Assignee: Unassigned
• Reporter: John Vines
• Created: 19/Nov/14 18:11
• Updated: 02/Nov/22 19:30
• Resolved: 02/Nov/22 19:30

Description

When a multi scan fails, for whatever reason, it gets double logged. This makes it harder to track down. We should settle on the logging coming from one location.

The locations in question are-
TabletServer.LookupTask.run, there's a catch Throwable that log.warns "exception while doing multi-scan " with the exception.

Then higher in the stack under
TabletServer.ThriftClientHandler.continueMultiScan, there's another catch Throwable that log.warns "Failed to get multiscan result"

Then higher in the stack under thrift in
ProcessFunction.process, there is another catch Throwable. This log.error "internal error processing *" and logs the exception.

We should
A. settle on multi-scan or multiscan or something else
B. decide which one we prefer
C. remove/mute the other ones in our default logging configuration