Details
-
Task
-
Status: Resolved
-
Medium
-
Resolution: Done
-
None
Description
This an overloaded construct, which serves multiple purposes:
- it is used by ParserIdentifier as a return value, serving all callers who wish to parse a request URL's resource identifier part
- is is used in NormalizedNodeContext as the context in which the result should be interpreted
- it is used intermittenly along the path between 1) and 2)
At the end of the day, with current state of refactoring, we are completely in control of the call sites that interact with 1) and are in a position resolution rules. For example, in POST we are able to say that 'parser this identifier as a datastore reference or a invocation reference'. These two results should be backed by separate records.
For part 2), we want to inline the inforation, which really should boil down to an Inference, into NormalizedNodeContext. This part may be plagued by legacy, so needs to be looked at carefully.
Transitions between the two should be easy to model and simple to translate, so 3) should be simple after the first two are dealt with.
Attachments
| # | Subject | Branch | Project | Status | CR | V |
|---|---|---|---|---|---|---|
| 107370,18 | RestconfServer requires ApiPath identifiers | master | netconf | Status: MERGED | +2 | +1 |
| 107845,4 | Disconnect PatchStatusContext from InstanceIdentifierContext | master | netconf | Status: MERGED | +2 | +1 |
| 107847,1 | Rename Patch{Context,Entity} | master | netconf | Status: ABANDONED | 0 | +1 |
| 107852,1 | Move InstanceIdentifierContext | master | netconf | Status: MERGED | +2 | +1 |
| 107853,1 | Remove InstanceIdentifierContext from PatchBody parsing | master | netconf | Status: MERGED | +2 | +1 |
| 107854,3 | Clean up JsonNormalizedNodeBodyWriter | master | netconf | Status: MERGED | +2 | +1 |
| 107855,3 | Clean up XmlNormalizedNodeBodyWriter | master | netconf | Status: MERGED | +2 | +1 |
| 107856,3 | Share common parts of AbstractNormalizedNodeBodyWriter | master | netconf | Status: MERGED | +2 | +1 |
| 107857,5 | Refactor AbstractNormalizedNodeBodyWriter | master | netconf | Status: MERGED | +2 | +1 |
| 107858,5 | Do not use InstanceIdentifierContext in NormalizedNodePayload | master | netconf | Status: MERGED | +2 | +1 |
| 107861,5 | Do not use InstanceIdentifierContext in OperationsContent | master | netconf | Status: MERGED | +2 | +1 |
| 107863,4 | Do not use InstanceIdentifierContext in postData() | master | netconf | Status: MERGED | +2 | +1 |
| 107865,4 | WIP: Add RequestUrl | master | netconf | Status: ABANDONED | 0 | -1 |
| 107893,10 | Create MdsalRestconfServer | master | netconf | Status: MERGED | +2 | +1 |
| 107894,12 | WIP: Introduce RequestMode | master | netconf | Status: ABANDONED | 0 | -1 |
| 109028,3 | Add ApiPath.toString() | master | netconf | Status: MERGED | +2 | +1 |
| 109030,6 | Eliminate space escaping | master | netconf | Status: MERGED | +2 | +1 |
| 109037,2 | Improve RestconfSchemaServiceTest | master | netconf | Status: MERGED | +2 | +1 |
| 109038,5 | Clean up ParserIdentifierTest | master | netconf | Status: MERGED | +2 | +1 |
| 109039,2 | Remove ParserIdentifier.makeQNameFromIdentifier() | master | netconf | Status: MERGED | +2 | +1 |
| 109057,4 | Fix JaxRsApiPath declaration | master | netconf | Status: MERGED | +2 | +1 |
| 109059,3 | Eliminate JaxRsApiPath | master | netconf | Status: MERGED | +2 | +1 |
| 109063,6 | Refactor modulesGET() methods | master | netconf | Status: MERGED | +2 | +1 |
| 109067,1 | Hide ParserIdentifier | master | netconf | Status: MERGED | +2 | +1 |
| 109069,3 | Disconnect WriterFieldsTranslator from InstanceIdentifierContext | master | netconf | Status: MERGED | +2 | +1 |
| 109070,16 | Eliminate InstanceIdentifierContext | master | netconf | Status: MERGED | +2 | +1 |
| 109071,2 | Add DatabindContext.schemaTree() | master | netconf | Status: MERGED | +2 | +1 |
| 109072,1 | WIP: Use DatabindContext for parsing | master | netconf | Status: ABANDONED | 0 | -1 |
| 109077,3 | Eliminate DOMDatabindProvider | master | netconf | Status: MERGED | +2 | +1 |
| 109078,4 | Use DatabindContext in InstanceIdentifierContext | master | netconf | Status: MERGED | +2 | +1 |
| 109079,2 | Remove InstanceIdentifierContext.getSchemaContext() | master | netconf | Status: MERGED | +2 | +1 |
| 109080,2 | Use DatabindContext in deserialization path | master | netconf | Status: MERGED | +2 | +1 |
| 109081,2 | Use DatabindContext in instance identifier serialization | master | netconf | Status: MERGED | +2 | +1 |
| 109084,3 | Introduce restconf.server.api.DataPostPath | master | netconf | Status: MERGED | +2 | +1 |
| 109087,3 | Add OperationsPostPath | master | netconf | Status: MERGED | +2 | +1 |
| 109088,1 | Introduce DataPutPath | master | netconf | Status: MERGED | +2 | +1 |
| 109089,1 | Add DataPatchPath | master | netconf | Status: MERGED | +2 | +1 |
| 109090,1 | Remove unused InstanceIdentifier factory methods | master | netconf | Status: MERGED | +2 | +1 |
| 109094,1 | Refactor YangInstanceIdentifierDeserializer | master | netconf | Status: MERGED | +2 | +1 |
| 109095,2 | Clean up YangInstanceIdentifierSerializer | master | netconf | Status: MERGED | +2 | +1 |
| 109097,1 | Eliminate IdentifierCodec | master | netconf | Status: MERGED | +2 | +1 |
| 109098,2 | Eliminate ParserIdentifier | master | netconf | Status: MERGED | +2 | +1 |
| 109099,1 | Clean up ParserIdentifierTest | master | netconf | Status: MERGED | +2 | +1 |
| 109128,4 | Generalize ResourceBodyTest | master | netconf | Status: MERGED | +2 | +1 |
| 109146,1 | Clean up exception handling | master | netconf | Status: MERGED | +2 | +1 |
| 109148,2 | Clean up data{DELETE,PATCH}() | master | netconf | Status: MERGED | +2 | +1 |