szetszwo commented on pull request #240: URL: https://github.com/apache/incubator-ratis/pull/240#issuecomment-719118827
For servers, they usually has a start() method. I guess servers may involve more resources so that the initialisation often takes a long time. It easy to manage to separate the initialisation and the start methods. e.g. io.grpc.Server has a start method. In contrast, clients are usually lightweight and connecting to a single server/service. It is a bigger change to remove the DataStreamServerRpc.start() method since RaftServer also has a start() method. We must make them consistent. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
