Skip to content

table: Adding getTableContent function#20

Merged
stcheng merged 1 commit intosonic-net:masterfrom
stcheng:table
May 8, 2016
Merged

table: Adding getTableContent function#20
stcheng merged 1 commit intosonic-net:masterfrom
stcheng:table

Conversation

@stcheng
Copy link
Copy Markdown
Contributor

@stcheng stcheng commented May 7, 2016

This function is used to retrieve the whole content of a table.

It will future be used for orchagent restarting logics.

common/table.cpp Outdated
}
}

bool Table::getTableKeys(std::vector<std::string> &keys)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

return could be void since if no elements then you will get empty result
you are also not checking output of this function

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

thanks. fixed.

@kcudnik
Copy link
Copy Markdown
Contributor

kcudnik commented May 8, 2016

ack

This function is used to retrieve the whole content of a table.
@stcheng stcheng merged commit 3f0ff1a into sonic-net:master May 8, 2016
@stcheng stcheng deleted the table branch May 8, 2016 08:31
prgeor pushed a commit to prgeor/sonic-swss-common that referenced this pull request Feb 27, 2025
Enable decode of QSFP-DD/OSFP transceivers, and vendor information.
Pterosaur pushed a commit to Pterosaur/sonic-swss-common that referenced this pull request Apr 13, 2025
Adding multi-db instance support based on db config file.
So, now instead of hardcoded DB endpoint, it is read from a configuration file.
nazariig pushed a commit to nazariig/sonic-swss-common that referenced this pull request May 6, 2025
…t#20)

```<br>* c7d30d3 - (HEAD -> 202412) Merge branch '202411' of https://github.com/sonic-net/sonic-swss-common into 202412 (2025-02-28) [Sonic Automation]
* 3bc4141 - (origin/202411) [FC] remove FLEX_COUNTER_DELAY_STATUS_FIELD (sonic-net#982) (2025-03-01) [mssonicbld]<br>```
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