Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CVL Sort issue in target TABLE with multi-list #135

Merged

Conversation

faraazbrcm
Copy link
Contributor

@faraazbrcm faraazbrcm commented Apr 8, 2024

When dependent target TABLE has multiple lists, if there exists a partial dependency, the loosely hanging LIST was causing sorting issue.

Below are the fix implemented

  1. Retained the last LIST entry instead of first.
  2. Injected dependency on all target LISTs of the TABLE.

When dependent target TABLE has multiple lists, if there exists a
partial dependency, the loosely hanging LIST was causing sorting issue.

Below are the fix implemented

1. Retained the last LIST entry instead of first.
2. Injected dependency on all target LISTs of the TABLE.
@faraazbrcm faraazbrcm force-pushed the singleton_sort_fixes branch from 44ceab7 to 3a87050 Compare April 8, 2024 23:22
@faraazbrcm faraazbrcm changed the title [JIRA SONIC-85996]-CVL Sort issue in target TABLE with multi-list CVL Sort issue in target TABLE with multi-list Apr 8, 2024
@kwangsuk kwangsuk self-requested a review April 10, 2024 18:16
@anand-kumar-subramanian anand-kumar-subramanian merged commit 72e0df9 into sonic-net:master Apr 15, 2024
2 checks passed
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.

3 participants