Details
-
Task
-
Status: Resolved
-
Medium
-
Resolution: Done
-
None
-
None
Description
org.opendaylight.restconf.nb.rfc8040.rests.utils contains standalone utilities which operate at various levels – mostly dealing with RestconfStrategy without the implication of a full RestconfDataService.
Eliminate individual classes here, replacing them with either RestconfDataServiceImpl integration (where JAX-RS is involved) or RestconfStrategy integration (where it is not).
Success criteria here are:
- org.opendaylight.restconf.nb.rfc8040.rests.utils package is gone
- RestconfDataServiceImpl is a plain HTTP wrapper around RestconfStrategy lookup and invocation
- RestconfStrategy and RestconfTransaction semantics are completely defined around either Netconf or DOMDataBroker
Attachments
There are no Sub-Tasks for this issue.
Gerrit Reviews
| # | Subject | Branch | Project | Status | CR | V |
|---|---|---|---|---|---|---|
| 107158,7 | Split up DeleteDataTransactionUtilTest | master | netconf | Status: MERGED | +2 | +1 |
| 107185,1 | Clean up {Post,Put}DataTransactionUtil.submitData() | master | netconf | Status: MERGED | +2 | +1 |
| 107196,2 | Move PutDataTransactionUtil.readList() | master | netconf | Status: MERGED | +2 | +1 |
| 107197,2 | Clean up testPostData() | master | netconf | Status: MERGED | +2 | +1 |
| 107198,1 | Evacuate static methods tests to RestconfDataServiceImplTest | master | netconf | Status: MERGED | +2 | +1 |
| 107199,1 | Eliminate PutDataTransactionUtilTest | master | netconf | Status: MERGED | +2 | +1 |
| 107200,1 | Clean up PlainPatchDataTransactionUtilTest | master | netconf | Status: MERGED | +2 | +1 |
| 107201,1 | Clean up PatchDataTransactionUtilTest | master | netconf | Status: MERGED | +2 | +1 |
| 107205,2 | Eliminate PostDataTransactionUtilTest | master | netconf | Status: MERGED | +2 | +1 |
| 107206,1 | Eliminate PlainPatchDataTransactionUtilTest | master | netconf | Status: MERGED | +2 | +1 |
| 107207,2 | Eliminate PatchDataTransactionUtilTest | master | netconf | Status: MERGED | +2 | +1 |
| 107208,2 | Clean up ReadDataTransactionUtilTest | master | netconf | Status: MERGED | +2 | +1 |
| 107209,2 | Eliminate ReadDataTransactionUtilTest | master | netconf | Status: MERGED | +2 | +1 |
| 107212,11 | Remove DeleteDataTransactionUtil | master | netconf | Status: MERGED | +2 | +1 |
| 107214,6 | Eliminate PlainPatchDataTransactionUtil | master | netconf | Status: MERGED | +2 | +1 |
| 107229,1 | Remove a stray FIXME | master | netconf | Status: MERGED | +2 | +1 |
| 107363,2 | Disconnect PutDataTransactionUtil from JAX-RS | master | netconf | Status: MERGED | +2 | +1 |
| 107366,4 | Update RestconfDataServiceImplTest assertions | master | netconf | Status: MERGED | +2 | +1 |
| 107367,3 | Disconnect PostDataTransactionUtil from JAX-RS | master | netconf | Status: MERGED | +2 | +1 |
| 107797,1 | Move CreateOrReplaceResult | master | netconf | Status: MERGED | +2 | +1 |
| 107799,2 | Eliminate PutDataTransactionUtil | master | netconf | Status: MERGED | +2 | +1 |
| 107800,3 | Restructure putData() implementation | master | netconf | Status: MERGED | +2 | +1 |
| 107801,3 | Eliminate PostDataTransactionUtil | master | netconf | Status: MERGED | +2 | +1 |
| 107803,1 | Optimize RestconfStrategy.postData() | master | netconf | Status: MERGED | +2 | +1 |
| 107805,1 | Hide RestconfTransaction.readList() | master | netconf | Status: MERGED | +2 | +1 |
| 107806,2 | Clean up PatchDataTransactionUtil | master | netconf | Status: MERGED | +2 | +1 |
| 107807,6 | Rehost PatchDataTransactionUtil logging | master | netconf | Status: MERGED | +2 | +1 |
| 107808,4 | Eliminate PatchDataTransactionUtil | master | netconf | Status: MERGED | +2 | +1 |
| 107815,4 | Remove extractReadData() | master | netconf | Status: MERGED | +2 | +1 |
| 107826,4 | Inline readAllData() | master | netconf | Status: MERGED | +2 | +1 |
| 107827,8 | Cross-relate WithDefaultsParam | master | netconf | Status: MERGED | +2 | +1 |
| 107832,3 | Eliminate (ReadData)TransactionUtil | master | netconf | Status: MERGED | +2 | +1 |
| 109164,2 | Repair testDeleteNonexistentData() | master | netconf | Status: MERGED | +2 | +1 |