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.

Sign up for GitHub

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

Jump to bottom

Tuning/threadpool exit #36

Merged
merged 5 commits into from
May 30, 2018
Merged

Tuning/threadpool exit #36

merged 5 commits into from
May 30, 2018

Conversation

xmtsui
Copy link
Member

@xmtsui xmtsui commented May 30, 2018

  1. 调整几个细节逻辑，启动失败后，主动调用一次stop方法确保状态一致，以及init资源的释放。
  2. 增加几个单测

@xmtsui xmtsui requested a review from yuemingliang May 30, 2018 03:55
@xmtsui xmtsui self-assigned this May 30, 2018
@xmtsui xmtsui added the dev:tuning Some improvement label May 30, 2018
@xmtsui xmtsui added this to the v1.4.2 release milestone May 30, 2018
@xmtsui xmtsui merged commit 97d65ac into release/1.4.2 May 30, 2018
xmtsui pushed a commit that referenced this pull request Jun 20, 2018
@yangl @xmtsui
[feature] support native transport (epoll) (#34)
4a8b9e8 
* (tuning) refactor start logic of remoting server

fix #8 (#21)

* tuning init logic of rpc server (#20)

fix #18

* (tuning) modify buffer allocator type

 fix #9 (#23)

* (format) modify header format (#24)

fix #10

* (tuning) log conf modify (#25)

fix #12  #13

* (format) modify license format

* fix maven-javadoc-plugin build ERROR with JDK 1.8 (#28)

* fix threadpools shutdown problems (issues 14) (#31)

单测待补充，先合并

* [feature] support the native transport

* merge feature/1.4.2

* [feature] support native transport (epoll)

* add autor

* make the bossGroup not deamon

* (bugfix) modify boss threadpool back to non-daemon

* [feature] support the native transport

* merge feature/1.4.2

* [feature] support native transport (epoll)

* add autor

* make the bossGroup not deamon

* rebase 1.4.2

* (feature) Netty epoll switch

* (version) modify release version

* Tuning/threadpool exit (#36)

* (tuning) change command factory visibility (#37)

* format error
@xmtsui xmtsui deleted the tuning/threadpool_exit branch June 27, 2018 04:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yuemingliang yuemingliang yuemingliang approved these changes

Assignees

@xmtsui xmtsui

Labels
dev:tuning Some improvement
Projects
None yet
Milestone
v1.4.2 release
Development

Successfully merging this pull request may close these issues.
2 participants
@xmtsui @yuemingliang