Parallelize GET and SUBSCRIBE processing#582
Open
niranjanivivek wants to merge 1 commit intosonic-net:masterfrom
Open
Parallelize GET and SUBSCRIBE processing#582niranjanivivek wants to merge 1 commit intosonic-net:masterfrom
niranjanivivek wants to merge 1 commit intosonic-net:masterfrom
Conversation
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
niranjanivivek
force-pushed
the
parallelize_get_subscribe_pr
branch
from
04087f3 to
9ab566f
Compare
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
niranjanivivek
force-pushed
the
parallelize_get_subscribe_pr
branch
from
9ab566f to
2c3143a
Compare
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Contributor
Author
|
@sreeiyer-1, @jayaragini-hcl Please review. |
niranjanivivek
force-pushed
the
parallelize_get_subscribe_pr
branch
from
2c3143a to
2ea6d04
Compare
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
niranjanivivek
force-pushed
the
parallelize_get_subscribe_pr
branch
from
2ea6d04 to
4b683c3
Compare
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
niranjanivivek
force-pushed
the
parallelize_get_subscribe_pr
branch
2 times, most recently
from
68532a9 to
ca77044
Compare
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
niranjanivivek
force-pushed
the
parallelize_get_subscribe_pr
branch
from
ca77044 to
44b3d7f
Compare
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Contributor
Author
|
@sreeiyer-1, Please review. |
Contributor
Author
ndas7
reviewed
Mar 7, 2026
niranjanivivek
force-pushed
the
parallelize_get_subscribe_pr
branch
from
44b3d7f to
132bf6f
Compare
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
ndas7
reviewed
Mar 12, 2026
gnmi_server/server_test.go
Outdated
| return | ||
| } | ||
| // Trigger an OnChange update. | ||
| if err = rclient.HSet(context.Background(), "DEVICE_METADATA|localhost", "config-meta-data", time.Now().String()).Err(); err != nil { |
Contributor
There was a problem hiding this comment.
This will need to be updated to hostname as well
Contributor
Author
There was a problem hiding this comment.
This is updated as hostname. L6343 is also updated with string hostname. Kindly confirm
ndas7
reviewed
Mar 12, 2026
niranjanivivek
force-pushed
the
parallelize_get_subscribe_pr
branch
from
132bf6f to
8bd7dd4
Compare
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
niranjanivivek
force-pushed
the
parallelize_get_subscribe_pr
branch
from
8bd7dd4 to
8f80b9b
Compare
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Signed-off-by: Niranjani Vivek <niranjaniv@google.com>
niranjanivivek
force-pushed
the
parallelize_get_subscribe_pr
branch
from
8f80b9b to
683a629
Compare
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Parallelization for gNMI Get() and Subscribe() operations. It improves performance, particularly when waiting for Redis responses.
Dependency PR:
Base BE PR to be merged first : Implement RCM - PR#202
Why I did it
How I did it
How to verify it
Which release branch to backport (provide reason below if selected)
Description for the changelog
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)