Re: GENIUS-289: migration of ManagedNewTransactionRunner


Hema Gopalakrishnan
 

Hi Robert,
Thanks for taking care of this. Will get the patches in as soon as the Silicon branch opens.
Thanks,
Hema

-----Original Message-----
From: TSC@... <TSC@...> On Behalf Of Robert Varga via lists.opendaylight.org
Sent: Tuesday, August 4, 2020 8:37 PM
To: app-dev@...
Cc: tsc@...
Subject: [OpenDaylight TSC] GENIUS-289: migration of ManagedNewTransactionRunner

Hello everyone,

ManagedNewTransactionRunner and related interfaces have been upstreamed to md-sal almost 2 years ago.

This effectively means that vast majority of

org.opendaylight.genius.infra

in mdsalutil-api duplicates functionality of

org.opendaylight.mdsal.binding.util

as available since at least mdsal-3.0.0 (i.e. for 18+ months now).


There are some subtle differences, which make this a transition which is not search&replace, I have therefore filed
https://protect2.fireeye.com/v1/url?k=d740fd5f-89e04731-d740bdc4-869a14f4b08c-66dfe688d7d24a24&q=1&e=698b518d-d621-4d0b-ae7d-7942637a9711&u=https%3A%2F%2Fjira.opendaylight.org%2Fbrowse%2FGENIUS-289 to track this.

That epic has a task for each and every component using ManagedNewTransactionRunner in both Genius and NetVirt, so that we get proper tracking of where we are with this transition.

I have filed patches for Genius to make the transition, but alas that needs to be synchronized with netvirt (at least where ITM and interfacemananger are concerned).

I would appreciate if the Genius committers could review the patches and get them merged as soon as stable/aluminium is branched (so that this work does not interfere with that release and lands in Silicon instead).

At the end of the day, this will eliminate some 4% of Genius code, easing the maintenance burden there.

Thanks,
Robert

Join {TSC@lists.opendaylight.org to automatically receive all group messages.