Bug 6787: FeatureConfigPusher confusing WARN log removed 41/53341/5
authorMichael Vorburger <vorburger@redhat.com>
Wed, 15 Mar 2017 12:33:09 +0000 (13:33 +0100)
committerTom Pantelis <tpanteli@brocade.com>
Tue, 21 Mar 2017 11:52:52 +0000 (11:52 +0000)
commitf5fcfe86c7baeea72b81886e01215840fbf1b91b
tree5fb2ff9dc8fc875cf51dec0a71e3228302288528
parent2b21b390c3417c98e48b93493fa77c248bbb1d60
Bug 6787: FeatureConfigPusher confusing WARN log removed

see analysis in bug: Current code already has a retry loop, and the
operation eventually succeeds; the ConcurrentModificationException
logged only created confusion and added no real value.

The code has been changed to log the last exception in the ERROR IFF
after N retries it does not succeed.

PS: We should create a generic utility helper for this ("Retryer").

Change-Id: I3c116e77f5a94366da15cc659b3a63d3a6e79f18
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
opendaylight/config/config-persister-feature-adapter/src/main/java/org/opendaylight/controller/configpusherfeature/internal/FeatureConfigPusher.java

©2013 OpenDaylight, A Linux Foundation Collaborative Project. All Rights Reserved.
OpenDaylight is a registered trademark of The OpenDaylight Project, Inc.
Linux Foundation and OpenDaylight are registered trademarks of the Linux Foundation.
Linux is a registered trademark of Linus Torvalds.