Optimize DataObjectCodecContext memory footprint 73/81873/1
authorRobert Varga <robert.varga@pantheon.tech>
Thu, 2 May 2019 10:27:58 +0000 (12:27 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Thu, 2 May 2019 11:12:42 +0000 (13:12 +0200)
commit680332443fe6e58b535985fc26a911e6f99d90f4
treea18529a15701a2fd87a143d2af55059373b3a094
parentd544e2a9235d269932219f294adbc87215ed1570
Optimize DataObjectCodecContext memory footprint

Examining heap dumps while working on MDSAL-442/MDSAL-443 revealed
a slight inefficiency in DataObjectCodecContext: we are forcing
byStreamClass's entrySet to be instantiated for the purposes of
copying it into byBindingArgClassBuilder.

This patch corrects the mistake by using byStreamClassBuilder
as the source of copied entries -- which holds the same content,
but we will be throwing it away.

Furthermore byStreamClass and byBindingArgClass are only distinct
when we have a choice child, hence we also run comparison on the
two builders and reuse byStreamClass if they are equal.

Change-Id: Ie58367f7cc899d49da7f78cd0d4e70fbb064ae99
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 09870ca2f1234be7c175667341f783c0b95ef408)
binding/mdsal-binding-dom-codec/src/main/java/org/opendaylight/mdsal/binding/dom/codec/impl/DataObjectCodecContext.java