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
toggle quoted message
Show quoted text
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
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