[NETCONF-1118] Refactor restconf.common.patch Created: 03/Aug/23  Updated: 12/Sep/23

Status: Confirmed
Project: netconf
Component/s: restconf-nb
Affects Version/s: None
Fix Version/s: 7.0.0

Type: Task Priority: Medium
Reporter: Robert Varga Assignee: Unassigned
Resolution: Unresolved Votes: 0
Labels: pt
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Blocks
is blocked by NETCONF-1128 Eliminate AbstractIdentifierAwareJaxR... Resolved
Epic Link: Rewire restconf-nb

 Description   

Patch utilities for RESTCONF have a number of down sides:

  • @Patch is defined here, making the entire package JAX-RS-dependent
  • naming should reflect the fact the binding is to NormalizedNode
  • some constructs (like PatchOperation) should be taken directly from binding-generated code
  • PatchEntity is really an edit
  • PatchContext is tied to InstanceIdentifierContext, making it hard to pass down to contexts which are not tied to it (while retaining patch-id, etc.)

 

 Comments   
Comment by Robert Varga [ 20/Aug/23 ]

As part of this issue we really should just eliminate the custom classes and let YANG-based XML/JSON codecs translate the data to NormalizedNode and then, for convenience, translate them to binding DTOs.

The problem with that is that we need to deal with target leaves – but that's okay, as we can deal with them based on DatabindContext and the target resouce YangInstanceIdentifier.

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