Lower tracing overhead 43/22243/2
authorRobert Varga <rovarga@cisco.com>
Tue, 9 Jun 2015 23:38:42 +0000 (01:38 +0200)
committerRobert Varga <rovarga@cisco.com>
Tue, 9 Jun 2015 23:46:05 +0000 (01:46 +0200)
commitc92a173cec44262a51e61d5bc3cc698aee6ed7bc
treef94924e485f8a5f5a802792c47ae7c0e969194b9
parentbcaeb7773976be48a2a8f52dd9fdf33e66d333a6
Lower tracing overhead

Arrays.toString() and ByteBufUtil.hexDump() are expensive relative to an
disabled trace call. Guard the corresponding trace() calls with
LOG.isTraceEnabled(). Discovered in BGP performance traces to have some
4% overhead.

Change-Id: I94ce9b57b86bae9be35e5839447a19b268064469
Signed-off-by: Robert Varga <rovarga@cisco.com>
bgp/parser-impl/src/main/java/org/opendaylight/protocol/bgp/parser/impl/message/BGPOpenMessageParser.java
bgp/parser-impl/src/main/java/org/opendaylight/protocol/bgp/parser/impl/message/BGPUpdateMessageParser.java
bgp/parser-impl/src/main/java/org/opendaylight/protocol/bgp/parser/impl/message/open/CapabilityParameterParser.java
bgp/rib-impl/src/main/java/org/opendaylight/protocol/bgp/rib/impl/BGPByteToMessageDecoder.java
bgp/rib-impl/src/main/java/org/opendaylight/protocol/bgp/rib/impl/BGPMessageToByteEncoder.java
bgp/rib-impl/src/main/java/org/opendaylight/protocol/bgp/rib/impl/LocRibWriter.java