93
72%

Ran 25 May 2018 02:18PM UTC

Jobs 1

Files 418

Run time 592min

Badge

Embed ▾

pending completion

Build # 93

Build Type

push

jenkins

Committed by Sijie Guo

Commit Message

[TABLE SERVICE] client interceptor and storage container grpc proxy

Descriptions of the changes in this PR:

This is a subsequent change after apache/bookkeeper#1428.

*Motivation*

Current almost every grpc requests are wrapped into `StorageContainerRequest` and their responses
are wrapped into `StorageContainerResponse`. It makes things a bit complicated on adding new grpc
services.

*Changes*

To simplify things, this PR introduces two functionalities for simplifying dispatching container requests/responses.

1) *StorageContainerClientInterceptor*: A grpc `ClientInterceptor` that stamps container information (currently is `scId`) into the requests' metadata before sending the requests to the wire.

2) A simple grpc reverse proxy to dispatch grpc requests to the channels provided by a `ChannelFinder`.

*Tests*

1. Existing unit tests covered client interceptor changes.
2. Introduced a `stream-storage-tests-common` module to include common classes that would be used for testing.
3. Introduced a `PingPongService` for testing reverse proxy : unary/client-streaming/server-streaming/bidi-streaming.

Master Issue: #1205

Author: Sijie Guo <sijie@apache.org>

Reviewers: Jia Zhai <None>

This closes #1430 from sijie/interceptor_container_requests

Run Details

24022 of 33105 relevant lines covered (72.56%)

0.73 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	93.1	25 May 2018 02:18PM UTC	0	72.56

Source Files on build 93

Detailed source file information is not available for this build.

apache / bookkeeper / 93
72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 93

apache / bookkeeper / 93 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 93

apache / bookkeeper / 93
72%

README BADGES
x