Skip to content

Disallow duplicate dyamic field names#134

Merged
anand-kumar-subramanian merged 1 commit intosonic-net:masterfrom
faraazbrcm:cvl_dynamic_field_names
Apr 8, 2024
Merged

Disallow duplicate dyamic field names#134
anand-kumar-subramanian merged 1 commit intosonic-net:masterfrom
faraazbrcm:cvl_dynamic_field_names

Conversation

@faraazbrcm
Copy link
Contributor

@faraazbrcm faraazbrcm commented Apr 8, 2024

CVL relying on 2-key list to determine the mapping Instead it should rely on one key and one non-key leaf

CVL relying on 2-key list to determine the mapping
Instead it should rely on one key and one non-key leaf

Change-Id: Ia26cb5d3f7f760e7899aa625e7cfc7d3a1e83035
Copy link
Contributor

@ranjinidn ranjinidn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested these changes with #129 code and it seems to work well. Hence approving these changes.

@anand-kumar-subramanian anand-kumar-subramanian merged commit 23d8bc1 into sonic-net:master Apr 8, 2024
@faraazbrcm faraazbrcm changed the title [JIRA SONIC-88436] - Disallow duplicate dyamic field names Disallow duplicate dyamic field names Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants