-
Notifications
You must be signed in to change notification settings - Fork 270
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
Add definition for MEMORY_STATISTICS table in schema #898
base: master
Are you sure you want to change the base?
Add definition for MEMORY_STATISTICS table in schema #898
Conversation
Signed-off-by: Arham-Nasir <[email protected]>
Signed-off-by: Arham-Nasir <[email protected]>
LGTM, pls resolve the conflicts. |
@FengPan-Frank , thank you for the review! I checked again. Please let me know if you need any further information. |
/azpw run |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
@xincunli-sonic @qiluo-msft @zbud-msft pls help review this PR. All swsscommon/db changes are getting failed since this PR in not Part of master |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me , please wait till all checks passes and merge this PR!
Hi @FengPan-Frank , can you please help merge this PR or get it merged with help of authorized person |
I don't have merge permission, either. pls check with qiluo-msft |
@qiluo-msft can you please help review this feature and PRs. |
define table in common/schema.h file of sonic-swss-common