Skip to content
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.

Sign up for GitHub

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

Jump to bottom

grpcbox-stream: Fix to support multiple simultaneous unary/strem calls #34

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

grpcbox-stream: Fix to support multiple simultaneous unary/strem calls #34

wants to merge 4 commits into from

Commits on Jul 2, 2020

  1. grpcbox-ct: Stress test grpcbox 

    Added a stress_test to stress number of concurrent gRPC sessions that can be run. BY default it is set to 10. But can be modified by environment variable GRPCBOX_STRESS_TEST

# To run 25 concurrent sessions:
$ GRPCBOX_STRESS_TEST=25 rebar3 ct --suite grpcbox_SUITE --case stress_test

On my system, it above command breaks around 30-35.

Signed-off-by: Vasu Dasari <[email protected]>
    @vasu-dasari
    vasu-dasari committed Jul 2, 2020
    Configuration menu
    Copy the full SHA
    93c4556 View commit details
    Browse the repository at this point in the history

Commits on Nov 8, 2020

  1. Merge branch 'master' of https://github.com/tsloughter/grpcbox into m… 

    …aster
    @vasu-dasari
    vasu-dasari committed Nov 8, 2020
    Configuration menu
    Copy the full SHA
    11943c4 View commit details
    Browse the repository at this point in the history

  2. grpcbox-stream: Add API is is_alive/1 

    Say there is a rRPC server offering "Streaming Output" service. And when client disconnects server side does not know about the client disconnection. This API offers a way for server to know if client socket is alive or not.

Signed-off-by: Vasu Dasari <[email protected]>
    @vasu-dasari
    vasu-dasari committed Nov 8, 2020
    Configuration menu
    Copy the full SHA
    3ad2a98 View commit details
    Browse the repository at this point in the history

Commits on Nov 16, 2020

  1. grpcbox-stream: Fix to support multiple simultaneous unary/strem calls 

    This commit addresses stress-test failure mentioned in tsloughter#29.

Added two new APIs:
add_channel(Name, Endpoints, Options)
delete_channel(Pid)

This would give ability to user to add and delete channels on the fly.

Also modified stress_test test case to use this logic. With out this change, stress test fails around 10 simultaneous connections. With this change I can see around 90 simultaneous connections.

Signed-off-by: Vasu Dasari <[email protected]>
    @vasu-dasari
    vasu-dasari committed Nov 16, 2020
    Configuration menu
    Copy the full SHA
    1ceee49 View commit details
    Browse the repository at this point in the history