Skip to content

Revert "Fix the issue of ignoring callback calls for removed keys. (#789)"#804

Merged
qiluo-msft merged 1 commit intomasterfrom
qiluo/revert789
Aug 1, 2023
Merged

Revert "Fix the issue of ignoring callback calls for removed keys. (#789)"#804
qiluo-msft merged 1 commit intomasterfrom
qiluo/revert789

Conversation

@qiluo-msft
Copy link
Contributor

This reverts commit e0f394c.

The reason is unexpected behavior change of get_entry. ref: #789 (comment)

@liuh-80
Copy link
Contributor

liuh-80 commented Jul 27, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80
Copy link
Contributor

liuh-80 commented Jul 28, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80
Copy link
Contributor

liuh-80 commented Jul 31, 2023

@oleksandrivantsiv , the PR #789 can't revert because vstest keeps failed, could you please check and fix to revert it?

@liuh-80
Copy link
Contributor

liuh-80 commented Aug 1, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@qiluo-msft qiluo-msft merged commit 7ff768a into master Aug 1, 2023
SviatoslavBoichuk pushed a commit to SviatoslavBoichuk/sonic-swss-common that referenced this pull request Sep 7, 2023
…onic-net#789)" (sonic-net#804)

This reverts commit e0f394c.

The reason is unexpected behavior change of `get_entry`. ref: sonic-net#789 (comment)
@qiluo-msft qiluo-msft deleted the qiluo/revert789 branch May 23, 2024 17:51
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.

2 participants