Rename to AsyncGeneric*

pull/1003/head
Yang Gao 10 years ago
parent 179f8684dc
commit 4999649653
  1. 14
      Makefile
  2. 4
      build.json
  3. 14
      include/grpc++/async_generic_service.h
  4. 9
      include/grpc++/generic_stub.h
  5. 12
      include/grpc++/server.h
  6. 6
      include/grpc++/server_builder.h
  7. 10
      src/cpp/server/async_generic_service.cc
  8. 10
      src/cpp/server/server.cc
  9. 6
      src/cpp/server/server_builder.cc
  10. 8
      test/cpp/end2end/generic_end2end_test.cc

@ -3108,7 +3108,7 @@ LIBGRPC++_SRC = \
src/cpp/common/completion_queue.cc \
src/cpp/common/rpc_method.cc \
src/cpp/proto/proto_utils.cc \
src/cpp/server/generic_service.cc \
src/cpp/server/async_generic_service.cc \
src/cpp/server/insecure_server_credentials.cc \
src/cpp/server/server.cc \
src/cpp/server/server_builder.cc \
@ -3121,6 +3121,7 @@ LIBGRPC++_SRC = \
src/cpp/util/time.cc \
PUBLIC_HEADERS_CXX += \
include/grpc++/async_generic_service.h \
include/grpc++/async_unary_call.h \
include/grpc++/byte_buffer.h \
include/grpc++/channel_arguments.h \
@ -3130,7 +3131,6 @@ PUBLIC_HEADERS_CXX += \
include/grpc++/config.h \
include/grpc++/create_channel.h \
include/grpc++/credentials.h \
include/grpc++/generic_service.h \
include/grpc++/impl/call.h \
include/grpc++/impl/client_unary_call.h \
include/grpc++/impl/internal_stub.h \
@ -3195,7 +3195,7 @@ src/cpp/common/call.cc: $(OPENSSL_DEP)
src/cpp/common/completion_queue.cc: $(OPENSSL_DEP)
src/cpp/common/rpc_method.cc: $(OPENSSL_DEP)
src/cpp/proto/proto_utils.cc: $(OPENSSL_DEP)
src/cpp/server/generic_service.cc: $(OPENSSL_DEP)
src/cpp/server/async_generic_service.cc: $(OPENSSL_DEP)
src/cpp/server/insecure_server_credentials.cc: $(OPENSSL_DEP)
src/cpp/server/server.cc: $(OPENSSL_DEP)
src/cpp/server/server_builder.cc: $(OPENSSL_DEP)
@ -3261,7 +3261,7 @@ $(OBJDIR)/$(CONFIG)/src/cpp/common/call.o:
$(OBJDIR)/$(CONFIG)/src/cpp/common/completion_queue.o:
$(OBJDIR)/$(CONFIG)/src/cpp/common/rpc_method.o:
$(OBJDIR)/$(CONFIG)/src/cpp/proto/proto_utils.o:
$(OBJDIR)/$(CONFIG)/src/cpp/server/generic_service.o:
$(OBJDIR)/$(CONFIG)/src/cpp/server/async_generic_service.o:
$(OBJDIR)/$(CONFIG)/src/cpp/server/insecure_server_credentials.o:
$(OBJDIR)/$(CONFIG)/src/cpp/server/server.o:
$(OBJDIR)/$(CONFIG)/src/cpp/server/server_builder.o:
@ -3352,7 +3352,7 @@ LIBGRPC++_UNSECURE_SRC = \
src/cpp/common/completion_queue.cc \
src/cpp/common/rpc_method.cc \
src/cpp/proto/proto_utils.cc \
src/cpp/server/generic_service.cc \
src/cpp/server/async_generic_service.cc \
src/cpp/server/insecure_server_credentials.cc \
src/cpp/server/server.cc \
src/cpp/server/server_builder.cc \
@ -3365,6 +3365,7 @@ LIBGRPC++_UNSECURE_SRC = \
src/cpp/util/time.cc \
PUBLIC_HEADERS_CXX += \
include/grpc++/async_generic_service.h \
include/grpc++/async_unary_call.h \
include/grpc++/byte_buffer.h \
include/grpc++/channel_arguments.h \
@ -3374,7 +3375,6 @@ PUBLIC_HEADERS_CXX += \
include/grpc++/config.h \
include/grpc++/create_channel.h \
include/grpc++/credentials.h \
include/grpc++/generic_service.h \
include/grpc++/impl/call.h \
include/grpc++/impl/client_unary_call.h \
include/grpc++/impl/internal_stub.h \
@ -3454,7 +3454,7 @@ $(OBJDIR)/$(CONFIG)/src/cpp/common/call.o:
$(OBJDIR)/$(CONFIG)/src/cpp/common/completion_queue.o:
$(OBJDIR)/$(CONFIG)/src/cpp/common/rpc_method.o:
$(OBJDIR)/$(CONFIG)/src/cpp/proto/proto_utils.o:
$(OBJDIR)/$(CONFIG)/src/cpp/server/generic_service.o:
$(OBJDIR)/$(CONFIG)/src/cpp/server/async_generic_service.o:
$(OBJDIR)/$(CONFIG)/src/cpp/server/insecure_server_credentials.o:
$(OBJDIR)/$(CONFIG)/src/cpp/server/server.o:
$(OBJDIR)/$(CONFIG)/src/cpp/server/server_builder.o:

@ -12,6 +12,7 @@
{
"name": "grpc++_base",
"public_headers": [
"include/grpc++/async_generic_service.h",
"include/grpc++/async_unary_call.h",
"include/grpc++/byte_buffer.h",
"include/grpc++/channel_arguments.h",
@ -21,7 +22,6 @@
"include/grpc++/config.h",
"include/grpc++/create_channel.h",
"include/grpc++/credentials.h",
"include/grpc++/generic_service.h",
"include/grpc++/impl/call.h",
"include/grpc++/impl/client_unary_call.h",
"include/grpc++/impl/internal_stub.h",
@ -57,7 +57,7 @@
"src/cpp/common/completion_queue.cc",
"src/cpp/common/rpc_method.cc",
"src/cpp/proto/proto_utils.cc",
"src/cpp/server/generic_service.cc",
"src/cpp/server/async_generic_service.cc",
"src/cpp/server/insecure_server_credentials.cc",
"src/cpp/server/server.cc",
"src/cpp/server/server_builder.cc",

@ -31,8 +31,8 @@
*
*/
#ifndef GRPCXX_GENERIC_SERVICE_H
#define GRPCXX_GENERIC_SERVICE_H
#ifndef GRPCXX_ASYNC_GENERIC_SERVICE_H
#define GRPCXX_ASYNC_GENERIC_SERVICE_H
#include <grpc++/byte_buffer.h>
#include <grpc++/stream.h>
@ -41,7 +41,7 @@ struct grpc_server;
namespace grpc {
typedef ServerAsyncReaderWriter<ByteBuffer, ByteBuffer> GenericServerReaderWriter;
typedef ServerAsyncReaderWriter<ByteBuffer, ByteBuffer> GenericServerAsyncReaderWriter;
class GenericServerContext GRPC_FINAL : public ServerContext {
public:
@ -55,14 +55,14 @@ class GenericServerContext GRPC_FINAL : public ServerContext {
grpc::string host_;
};
class GenericService GRPC_FINAL {
class AsyncGenericService GRPC_FINAL {
public:
// TODO(yangg) Once we can add multiple completion queues to the server
// in c core, add a CompletionQueue* argument to the ctor here.
GenericService() : server_(nullptr) {}
AsyncGenericService() : server_(nullptr) {}
void RequestCall(GenericServerContext* ctx,
GenericServerReaderWriter* reader_writer,
GenericServerAsyncReaderWriter* reader_writer,
CompletionQueue* cq, void* tag);
private:
@ -72,4 +72,4 @@ class GenericService GRPC_FINAL {
} // namespace grpc
#endif // GRPCXX_GENERIC_SERVICE_H
#endif // GRPCXX_ASYNC_GENERIC_SERVICE_H

@ -39,16 +39,19 @@
namespace grpc {
typedef ClientAsyncReaderWriter<ByteBuffer, ByteBuffer> GenericClientReaderWriter;
typedef ClientAsyncReaderWriter<ByteBuffer, ByteBuffer>
GenericClientAsyncReaderWriter;
// Generic stubs provide a type-unsafe interface to call gRPC methods
// by name.
class GenericStub GRPC_FINAL {
public:
explicit GenericStub(std::shared_ptr<ChannelInterface> channel) : channel_(channel) {}
explicit GenericStub(std::shared_ptr<ChannelInterface> channel)
: channel_(channel) {}
// begin a call to a named method
std::unique_ptr<GenericClientReaderWriter> Call(ClientContext* context, const grpc::string& method);
std::unique_ptr<GenericClientAsyncReaderWriter> Call(
ClientContext* context, const grpc::string& method);
private:
std::shared_ptr<ChannelInterface> channel_;

@ -50,7 +50,7 @@ struct grpc_server;
namespace grpc {
class AsynchronousService;
class GenericServerContext;
class GenericService;
class AsyncGenericService;
class RpcService;
class RpcServiceMethod;
class ServerCredentials;
@ -73,7 +73,7 @@ class Server GRPC_FINAL : private CallHook,
CompletionQueue* cq() { return &cq_; }
private:
friend class GenericService;
friend class AsyncGenericService;
friend class ServerBuilder;
class SyncRequest;
@ -86,7 +86,7 @@ class Server GRPC_FINAL : private CallHook,
// The service must exist for the lifetime of the Server instance.
bool RegisterService(RpcService* service);
bool RegisterAsyncService(AsynchronousService* service);
void RegisterGenericService(GenericService* service);
void RegisterAsyncGenericService(AsyncGenericService* service);
// Add a listening port. Can be called multiple times.
int AddPort(const grpc::string& addr, ServerCredentials* creds);
// Start the server.
@ -104,9 +104,9 @@ class Server GRPC_FINAL : private CallHook,
ServerAsyncStreamingInterface* stream,
CompletionQueue* cq, void* tag) GRPC_OVERRIDE;
void RequestGenericCall(GenericServerContext* context,
ServerAsyncStreamingInterface* stream,
CompletionQueue* cq, void* tag);
void RequestAsyncGenericCall(GenericServerContext* context,
ServerAsyncStreamingInterface* stream,
CompletionQueue* cq, void* tag);
// Completion queue.
CompletionQueue cq_;

@ -41,9 +41,9 @@
namespace grpc {
class AsyncGenericService;
class AsynchronousService;
class CompletionQueue;
class GenericService;
class RpcService;
class Server;
class ServerCredentials;
@ -66,7 +66,7 @@ class ServerBuilder {
void RegisterAsyncService(AsynchronousService* service);
// Register a generic service.
void RegisterGenericService(GenericService* service);
void RegisterAsyncGenericService(AsyncGenericService* service);
// Add a listening port. Can be called multiple times.
void AddPort(const grpc::string& addr,
@ -91,7 +91,7 @@ class ServerBuilder {
std::vector<AsynchronousService*> async_services_;
std::vector<Port> ports_;
std::shared_ptr<ServerCredentials> creds_;
GenericService* generic_service_;
AsyncGenericService* generic_service_;
ThreadPoolInterface* thread_pool_;
};

@ -31,16 +31,16 @@
*
*/
#include <grpc++/generic_service.h>
#include <grpc++/async_generic_service.h>
#include <grpc++/server.h>
namespace grpc {
void GenericService::RequestCall(GenericServerContext* ctx,
GenericServerReaderWriter* reader_writer,
CompletionQueue* cq, void* tag) {
server_->RequestGenericCall(ctx, reader_writer, cq, tag);
void AsyncGenericService::RequestCall(
GenericServerContext* ctx, GenericServerAsyncReaderWriter* reader_writer,
CompletionQueue* cq, void* tag) {
server_->RequestAsyncGenericCall(ctx, reader_writer, cq, tag);
}
} // namespace grpc

@ -39,7 +39,7 @@
#include <grpc/support/alloc.h>
#include <grpc/support/log.h>
#include <grpc++/completion_queue.h>
#include <grpc++/generic_service.h>
#include <grpc++/async_generic_service.h>
#include <grpc++/impl/rpc_service_method.h>
#include <grpc++/impl/service_type.h>
#include <grpc++/server_context.h>
@ -228,7 +228,7 @@ bool Server::RegisterAsyncService(AsynchronousService* service) {
return true;
}
void Server::RegisterGenericService(GenericService* service) {
void Server::RegisterAsyncGenericService(AsyncGenericService* service) {
GPR_ASSERT(service->server_ == nullptr &&
"Can only register an generic service against one server.");
service->server_ = this;
@ -395,9 +395,9 @@ void Server::RequestAsyncCall(void* registered_method, ServerContext* context,
new AsyncRequest(this, registered_method, context, request, stream, cq, tag);
}
void Server::RequestGenericCall(GenericServerContext* context,
ServerAsyncStreamingInterface* stream,
CompletionQueue* cq, void* tag) {
void Server::RequestAsyncGenericCall(GenericServerContext* context,
ServerAsyncStreamingInterface* stream,
CompletionQueue* cq, void* tag) {
new AsyncRequest(this, context, stream, cq, tag);
}

@ -52,10 +52,10 @@ void ServerBuilder::RegisterAsyncService(AsynchronousService* service) {
async_services_.push_back(service);
}
void ServerBuilder::RegisterGenericService(GenericService* service) {
void ServerBuilder::RegisterAsyncGenericService(AsyncGenericService* service) {
if (generic_service_) {
gpr_log(GPR_ERROR,
"Adding multiple GenericService is unsupported for now. "
"Adding multiple AsyncGenericService is unsupported for now. "
"Dropping the service %p", service);
return;
}
@ -96,7 +96,7 @@ std::unique_ptr<Server> ServerBuilder::BuildAndStart() {
}
}
if (generic_service_) {
server->RegisterGenericService(generic_service_);
server->RegisterAsyncGenericService(generic_service_);
}
for (auto& port : ports_) {
int r = server->AddPort(port.addr, port.creds.get());

@ -37,7 +37,7 @@
#include "test/core/util/test_config.h"
#include "test/cpp/util/echo.pb.h"
#include "src/cpp/util/time.h"
#include <grpc++/generic_service.h>
#include <grpc++/async_generic_service.h>
#include <grpc++/async_unary_call.h>
#include <grpc++/byte_buffer.h>
#include <grpc++/channel_arguments.h>
@ -87,7 +87,7 @@ class GenericEnd2endTest : public ::testing::Test {
// Setup server
ServerBuilder builder;
builder.AddPort(server_address_.str(), InsecureServerCredentials());
builder.RegisterGenericService(&generic_service_);
builder.RegisterAsyncGenericService(&generic_service_);
server_ = builder.BuildAndStart();
}
@ -124,7 +124,7 @@ class GenericEnd2endTest : public ::testing::Test {
ClientContext cli_ctx;
GenericServerContext srv_ctx;
GenericServerReaderWriter stream(&srv_ctx);
GenericServerAsyncReaderWriter stream(&srv_ctx);
send_request.set_message("Hello");
std::unique_ptr<ClientAsyncResponseReader<EchoResponse> > response_reader(
@ -171,7 +171,7 @@ class GenericEnd2endTest : public ::testing::Test {
CompletionQueue srv_cq_;
std::unique_ptr<grpc::cpp::test::util::TestService::Stub> stub_;
std::unique_ptr<Server> server_;
GenericService generic_service_;
AsyncGenericService generic_service_;
std::ostringstream server_address_;
};

Loading…
Cancel
Save