mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chun-Hung Hsiao <chhs...@mesosphere.io>
Subject Re: Review Request 61098: Added unit tests for gRPC support in libprocess.
Date Wed, 02 Aug 2017 21:51:10 GMT


> On July 28, 2017, 7:57 p.m., Jie Yu wrote:
> > 3rdparty/libprocess/src/tests/grpc_tests.cpp
> > Lines 143 (patched)
> > <https://reviews.apache.org/r/61098/diff/4/?file=1784965#file1784965line143>
> >
> >     I would separate discard into 3 tests:
> >     1) discard before server is started
> >     2) discard while server is processing the rpc
> >     3) discard after server finish processing the rpc
> >     
> >     maybe for 3), the expected result is a success?

For 3), I don't think there's a way to properly synchronize the statements to discard the
future after the server finishes processing the call and before the client runtime receives
the response.


- Chun-Hung


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61098/#review181712
-----------------------------------------------------------


On July 28, 2017, 12:49 a.m., Chun-Hung Hsiao wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61098/
> -----------------------------------------------------------
> 
> (Updated July 28, 2017, 12:49 a.m.)
> 
> 
> Review request for mesos, Benjamin Mahler, Jie Yu, and Joseph Wu.
> 
> 
> Bugs: MESOS-7810
>     https://issues.apache.org/jira/browse/MESOS-7810
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> We tested the following 6 scenarios when making gRPC calls:
> 
> 1. The server responds with an OK.
> 2. The server responds with a failure.
> 3. The client discards the calls.
> 4. The client makes mulitple outstanding calls.
> 5. The client shuts down during a call.
> 6. The server is unreachable.
> 7. The server dose not respond in time.
> 
> `grpc_tests.pb.h`, `grpc_tests.pb.cc`, `grpc_tests.grpc.pb.h` and
> `grpc_tests.grpc.pb.cc` are created with protobuf 3.3.0 and grpc
> 1.4.2 through the following command, assuming the current working
> directory is `3rdparty/libprocess/src/tests`:
> $ /path/to/protoc -I. --cpp_out=. --grpc_out=. \
>   --plugin=protoc-gen-grpc=/path/to/grpc_cpp_plugin \
>   grpc_tests.proto
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/src/tests/grpc_tests.cpp PRE-CREATION 
>   3rdparty/libprocess/src/tests/grpc_tests.grpc.pb.h PRE-CREATION 
>   3rdparty/libprocess/src/tests/grpc_tests.grpc.pb.cc PRE-CREATION 
>   3rdparty/libprocess/src/tests/grpc_tests.pb.h PRE-CREATION 
>   3rdparty/libprocess/src/tests/grpc_tests.pb.cc PRE-CREATION 
>   3rdparty/libprocess/src/tests/grpc_tests.proto PRE-CREATION 
> 
> 
> Diff: https://reviews.apache.org/r/61098/diff/4/
> 
> 
> Testing
> -------
> 
> N/A
> 
> 
> Thanks,
> 
> Chun-Hung Hsiao
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message