Skip to content

Remove duplicate open operation for swss.rec file#299

Closed
jipanyang wants to merge 1 commit intosonic-net:v1.0.3from
jipanyang:swssrec_fix
Closed

Remove duplicate open operation for swss.rec file#299
jipanyang wants to merge 1 commit intosonic-net:v1.0.3from
jipanyang:swssrec_fix

Conversation

@jipanyang
Copy link
Contributor

swss.rec has been opened in main(), should not be opened again here.

@msftclas
Copy link

@JipanYanga,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@jipanyang
Copy link
Contributor Author

Fixed in #298

@jipanyang jipanyang closed this Aug 25, 2017
@jipanyang jipanyang deleted the swssrec_fix branch June 2, 2018 02:04
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-swss that referenced this pull request Mar 1, 2023
…#299)

* [flex_counter]: Include plugins delay into poll correction

Signed-off-by: marian-pritsak <marianp@mellanox.com>

* Address comments

Signed-off-by: marian-pritsak <marianp@mellanox.com>

* Remove pollInterval parameter

Signed-off-by: marian-pritsak <marianp@mellanox.com>
lukasstockner pushed a commit to genesiscloud/sonic-swss that referenced this pull request Apr 2, 2023
* catch Exception to avoid CMIS code crash

Signed-off-by: Kebo Liu <kebol@nvidia.com>

* fix review comments, add more UT test

Signed-off-by: Kebo Liu <kebol@nvidia.com>

Signed-off-by: Kebo Liu <kebol@nvidia.com>
jianyuewu pushed a commit to jianyuewu/sonic-swss that referenced this pull request Dec 24, 2025
Indentation changes
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