Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Default bootstrap of consumer side. #90

Closed
ujjboy opened this issue May 5, 2018 · 0 comments
Closed

Default bootstrap of consumer side. #90

ujjboy opened this issue May 5, 2018 · 0 comments
Labels
enhancement New feature or request
Milestone

Comments

@ujjboy
Copy link
Member

ujjboy commented May 5, 2018

The default bootstrap is bolt now, if we use the other protocol, we must specified bootstrap like code below:

consumerConfig.setProtocol("http").setBootstrap("http");

It is not friendly. I will do some refactor:

  • Change default bootstrap extension to sofa (com.alipay.sofa.rpc.bootstrap.DefaultProviderBootstrap and com.alipay.sofa.rpc.bootstrap.DefaultConsumerBootstrap)
  • Use protocol as bootstrap name If the bootstrap of consumer is not specified. If not exist extension named it, then use default bootstrap.
@ujjboy ujjboy added the enhancement New feature or request label May 5, 2018
@ujjboy ujjboy added this to the 5.4.0 milestone May 5, 2018
@ujjboy ujjboy closed this as completed May 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant