You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2018/05/28 08:50:31 UTC

[GitHub] sijie opened a new pull request #1448: [table service] use grpc reverse proxy to serve storage container requests

sijie opened a new pull request #1448: [table service] use grpc reverse proxy to serve storage container requests
URL: https://github.com/apache/bookkeeper/pull/1448
 
 
   Descriptions of the changes in this PR:
   
   *Motivation*
   
   #1430 introduced client interceptor and grpc reverse proxy for storage container requests. This PR is the subsequent change to leverage client interceptor and reverse proxy for storage container requests.
   
   *Changes*
   
   **Client Changes**
   
   Changed `StorageContainerChannel` to add client interceptor to stamp storage container id into the the requests' metadata.
   
   **Server Changes**
   
   - moved the stream storage logic out of storage containers to a `service` package. the main logic will be kept in `RangeStoreService` and `RangeStoreServiceFactory`.
   - make the storage container logic for hosting `StorageContainerService`.
   - Each storage container will run an inprogress grpc server for serving the grpc services registered to the container and provide an `channel` for accessing those grpc service.
   - Changed the server to use reverse proxy for serving/proxying storage container requests.
   
   *NOTE*
   
   This change doesn't directly remove `StorageContainerRequest` and `StorageContainerResponse`.  It would be done in a subsequent change.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services