ygot related enhancements and fixes #34
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
mbalachandar
commented
Oct 30, 2020
mbalachandar
commented
- Enhanced the unmarshaling method to update the existing list entries instead of replacing it, and by providing this as one more option in the method.
- Optimized the EmitJson method to improve the performance of the Marshaling of object into json data
- Fixed the issue in finding the type, if the type is defined as typedf which got included as part of Include statement.
- Added the support to synchronize the access of the pathToSchema, ChildSchema, regexp maps
- Fixed the issue in MergeStruct method by comparing with src. object and updating the content of the dest. object properly
- Added the lint:ignore statment to avoid static check on the methods getNodeName, getObjectFieldName
- Fixed the issue to only skip the validation for sonic yang model, since the validation is handled in the CVL.
…e the performance
…ion and also added the goyang patches
…ion and also added the goyang patches
dell-engops
pushed a commit
to project-arlo/sonic-mgmt-common
that referenced
this pull request
Feb 7, 2022
…c_share-to-dell_sonic_share Merge broadcom_sonic_share to dell_sonic_share - 0203
kwangsuk
pushed a commit
that referenced
this pull request
Oct 2, 2024
* Internal Review: PortChannel support for REST and subscription (#33) * Support PortChannel, Subscription, Test * support portchannel subscription code --------- Co-authored-by: Satoru-Shinohara <satoru.shinohara@dell.com> Co-authored-by: Nikita Agarwal <nikita_agarwal1@dell.com> * Address Review Comments on Community PR (#34) --------- Co-authored-by: Nikita Agarwal <nikita_agarwal1@dell.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.