[#GENIUS-189] Enforce use of (only) managed transactions in mdsalutils

Convert all of genius to managed transactions (datastore-constrained) (GENIUS-176) [GENIUS-189] Enforce use of (only) managed transactions in mdsalutils Created: 25/Jul/18 Updated: 09/Aug/18 Resolved: 09/Aug/18
Status:	Resolved
Project:	genius
Component/s:	None
Affects Version/s:	None
Fix Version/s:	Fluorine

Type:

Sub-task

Priority:

Medium

Reporter:

Michael Vorburger

Assignee:

Edwin Anthony

Resolution:

Done

Votes:

Labels:

None

Remaining Estimate:

0 minutes

Time Spent:

1 day

Original Estimate:

Not Specified

Description

NB skitt's https://git.opendaylight.org/gerrit/#/c/73041/

Comments

Comment by Edwin Anthony [ 07/Aug/18 ]

currently the deprecated APIs cannot be removed entirely, because of their prolific usage.

a wrapper is introduced https://git.opendaylight.org/gerrit/#/c/74887/ to returned checked futures, which is used transaction runner's fluent future.
applications using the deprecated APIs

would see a performance hit because of the conversions introduced.

consumers of mdsalmanager are advised to migrate to the newer APIs at the earliest.

Comment by Michael Vorburger [ 07/Aug/18 ]

https://git.opendaylight.org/gerrit/#/c/74837/ just merged.

Not sure I understand what performance hit because of the conversion introduced you mean. If it's just the fact that we convert futures, that should be really fast.

Shall we close this issue now, or is there any other work to be done here?

Comment by Edwin Anthony [ 09/Aug/18 ]

oooh, yes, now I get it.
sure, I'll close it.
Thanks.

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

[GENIUS-189] Enforce use of (only) managed transactions in mdsalutils Created: 25/Jul/18 Updated: 09/Aug/18 Resolved: 09/Aug/18