[HBASE-4810] Clean up IPC logging configuration

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Incomplete
• Component/s: IPC/RPC
• Assignee: Unassigned
• Reporter: Gary Helmling
• Created: 17/Nov/11 18:15
• Updated: 12/Jun/22 19:43
• Resolved: 11/Apr/15 01:11

Description

The current IPC classes – HBaseClient, HBaseServer, HBaseRPC, WritableRpcEngine, etc. – use mangled package names (org.apache.hadoop.ipc) when obtaining loggers so that we can enable "debug" logging on the full org.apache.hadoop.hbase base package and not have the noise from per-request log messages drown out other important information. This has the desired effect, but is extremely hacky and counter-intuitive.

I think it would be better to fix the package name used by these classes (use org.apache.hadoop.hbase.ipc), and change the noisy, per-request messages to be logged at "trace" level. This will keep those messages out of standard debug output, while allowing the log4j configuration when you actually wish to see them to be a little more intuitive.

Issue Links

• is related to: HBASE-2182 rpc/ipc refactor; x-version compatibility, nio, async, enveloping, sane timeouts, etc.