add additional (debug) logging to AbstractStreamWriterGenerator 49/71849/4
authorMichael Vorburger <vorburger@redhat.com>
Tue, 8 May 2018 14:59:24 +0000 (16:59 +0200)
committerMichael Vorburger <vorburger@redhat.com>
Wed, 9 May 2018 11:54:40 +0000 (13:54 +0200)
commitd6f766e56a8acf87bafdb1a97791bf3400636f9a
treed13e20740df7ac6a2beef976c429493f09b4ac94
parentebad4b233a0f77232b48e4c1958ca0c15c132930
add additional (debug) logging to AbstractStreamWriterGenerator

* LOG.debug() in generateSerializer() to see if in a load test
scenario that ever gets called twice for the same type (it should not)
due to a concurrency bug I'm not seeing.

* reduce the only other LOG.debug() in that class to trace()
because that one will happen more often (constantly?) and make it hard
to see the other one.

* log onBindingRuntimeContextUpdated() on the off chance
that changing the BindingRuntimeContext somehow could cause this
suspected concurrency issue.

* log constructor to better understand the lifecycle of
AbstractStreamWriterGenerator in production.

* add Guava Cache removalListener with logging to see if this is caused
by eviction by GC due to .weakKeys() - ACK to tpantelis for this hunch.

JIRA: MDSAL-213
Change-Id: I2416131fe140ee483cc93740480ee0e37f0ce130
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
binding/mdsal-binding-dom-codec/src/main/java/org/opendaylight/mdsal/binding/dom/codec/gen/impl/AbstractStreamWriterGenerator.java