[GENIUS-176] Convert all of genius to managed transactions (datastore-constrained) Created: 25/Jul/18  Updated: 04/Feb/19

Status: In Progress
Project: genius
Component/s: None
Affects Version/s: None
Fix Version/s: Neon

Type: Improvement Priority: Medium
Reporter: Michael Vorburger Assignee: Faseela K
Resolution: Unresolved Votes: 0
Labels: None
Σ Remaining Estimate: 0 minutes Remaining Estimate: Not Specified
Σ Time Spent: 1 day Time Spent: Not Specified
Σ Original Estimate: Not Specified Original Estimate: Not Specified

Issue Links:
Blocks
blocks NETVIRT-1318 MDSAL best practices In Progress
is blocked by GENIUS-102 New OOM due to more TX leaks seen in ... Resolved
is blocked by GENIUS-178 Fix new transaction leaks in idmanager Resolved
Relates
relates to NETVIRT-1384 Umbrella: Numerous new transaction leaks Resolved
Sub-Tasks:
Key
 Summary
 Type
 Status
 Assignee
GENIUS-177 Convert interfacemanger to managed tr... Sub-task Resolved Stephen Kitt  
GENIUS-179 Enforce use of (only) managed transac... Sub-task Resolved Michael Vorburger  
GENIUS-180 Enforce use of (only) managed transac... Sub-task In Progress Nobin Mathew  
GENIUS-181 Enforce use of (only) managed transac... Sub-task Resolved Faseela K  
GENIUS-182 Enforce use of (only) managed transac... Sub-task Resolved Faseela K  
GENIUS-183 Enforce use of (only) managed transac... Sub-task Resolved Michael Vorburger  
GENIUS-184 Enforce use of (only) managed transac... Sub-task Verified Faseela K  
GENIUS-185 Enforce use of (only) managed transac... Sub-task Resolved Edwin Anthony  
GENIUS-186 Enforce use of (only) managed transac... Sub-task Verified Faseela K  
GENIUS-187 Enforce use of (only) managed transac... Sub-task Resolved Faseela K  
GENIUS-188 Enforce use of (only) managed transac... Sub-task Resolved Faseela K  
GENIUS-189 Enforce use of (only) managed transac... Sub-task Resolved Edwin Anthony  

 Description   

Because of NETVIRT-1384, we want to eventually completely forbid the direct usage of the DataBroker API in all of genius, and enforce all code to only ever use the utilities such as datastore-constrained transactions which guarantee that there are no transaction leaks.

This is basically the genius equivalent of what NETVIRT-1318 did for netvirt. - The short term priority is on doing this for what is due to genius bugs in NETVIRT-1384 (see linked issues).

Ultimately we want to be able to enforce this in all of genius like skitt has already done for netvirt (see the  databroker-checks.xml Checkstyle configuration and its use in the netvirt/commons/binding-parent/pom.xml).

 Comments   
Comment by Faseela K [ 25/Jul/18 ]

vorburger : Are all these cherry-picks from master? Or do you want me to assign these tasks to the module owners and get it done?

Comment by Michael Vorburger [ 25/Jul/18 ]

This is about new to be done work (thus Enhancement), not cherry-picks; yes assign to module owners would be a good start. The first two sub-tasks (GENIUS-177 & GENIUS-179) should illustrate how to do this.

Comment by Michael Vorburger [ 04/Feb/19 ]

k.faseela may I re-assign this issue to you to track and close when fully done? Just 1 sub-task is left.

Generated at Wed Feb 07 20:00:06 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.