Code Review / genius.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: dc7a685) | patch
Fix Dependency Injection IdManager appears to have to have a LockManager 98/43998/2
authorMichael Vorburger <vorburger@redhat.com>
Mon, 15 Aug 2016 15:56:52 +0000 (17:56 +0200)
committerMichael Vorburger <vorburger@redhat.com>
Tue, 16 Aug 2016 10:57:02 +0000 (12:57 +0200)
commit42bb55b990a05ca0f262a10ff41d5d62afc93378
treea2ebdf0c1c09d92481453d4436d9e61eeb3711a0tree | snapshot
parentdc7a6853b6ec5e26b35150481ae98891d7dddc3fcommit | diff
Fix Dependency Injection IdManager appears to have to have a LockManager

Therefore it's constructor should expect one, just like it does for
DataBroker (AKA "constructor injection") instead of having a separate
method for it (AKA "setter injection").

Unless a LockManager is optional for an IdManager, which doesn't seem to
be the case.

Change-Id: I33a3d0a0f0ec7a2899ec8febdceef4a593d4cbd1
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
idmanager/idmanager-impl/src/main/java/org/opendaylight/genius/idmanager/IdManager.java diff | blob | history
Genius