Skip to content

Optimize state_db update into batch way.#176

Merged
qiluo-msft merged 1 commit intosonic-net:masterfrom
FengPan-Frank:fix_db
Oct 31, 2024
Merged

Optimize state_db update into batch way.#176
qiluo-msft merged 1 commit intosonic-net:masterfrom
FengPan-Frank:fix_db

Conversation

@FengPan-Frank
Copy link
Contributor

@FengPan-Frank FengPan-Frank commented Oct 31, 2024

Optimize state_db update into batch way.

Work item tracking
Microsoft ADO (number only):30159330

test result:
image

@qiluo-msft qiluo-msft merged commit f95b7cd into sonic-net:master Oct 31, 2024
mssonicbld added a commit to mssonicbld/sonic-host-services that referenced this pull request Jul 3, 2025
What I did
Fix ProcessStatsST column issue

Why I did it
sonic-net#176, when did optimization of the state_db update into batch as dict, not realized "%" is missed, which is a regression issue.
![image](https://github.com/user-attachments/assets/b544ce77-9670-479b-822f-5b7ca74192ba)

How I verified it
Correct typo and added corresponding unit test to cover column name check.
mssonicbld added a commit to mssonicbld/sonic-host-services that referenced this pull request Jul 10, 2025
What I did
Fix ProcessStatsST column issue

Why I did it
sonic-net#176, when did optimization of the state_db update into batch as dict, not realized "%" is missed, which is a regression issue.
![image](https://github.com/user-attachments/assets/b544ce77-9670-479b-822f-5b7ca74192ba)

How I verified it
Correct typo and added corresponding unit test to cover column name check.
mssonicbld added a commit that referenced this pull request Jul 10, 2025
…#286)

What I did
Fix ProcessStatsST column issue

Why I did it
#176, when did optimization of the state_db update into batch as dict, not realized "%" is missed, which is a regression issue.
![image](https://github.com/user-attachments/assets/b544ce77-9670-479b-822f-5b7ca74192ba)

How I verified it
Correct typo and added corresponding unit test to cover column name check.
mssonicbld added a commit that referenced this pull request Jul 11, 2025
…#281)

What I did
Fix ProcessStatsST column issue

Why I did it
#176, when did optimization of the state_db update into batch as dict, not realized "%" is missed, which is a regression issue.
![image](https://github.com/user-attachments/assets/b544ce77-9670-479b-822f-5b7ca74192ba)

How I verified it
Correct typo and added corresponding unit test to cover column name check.
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