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

fix threadpools shutdown problems (issues 14) #31

Merged
merged 6 commits into from
May 28, 2018
Merged

fix threadpools shutdown problems (issues 14) #31

merged 6 commits into from
May 28, 2018

Conversation

yangl
Copy link
Contributor

@yangl yangl commented May 22, 2018

No description provided.

*/
@Override
public void shutdown() {
workerGroup.shutdownGracefully();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

workerGroup 应该是全局的,在一个非static方法里关掉会不会有问题?

@yangl
Copy link
Contributor Author

yangl commented May 22, 2018

@ujjboy workerGroup静态变量问题已修复

@xmtsui
Copy link
Member

xmtsui commented May 22, 2018

@yangl format的问题,麻烦再提交一下哈。现在check format还是不过

@xmtsui
Copy link
Member

xmtsui commented May 22, 2018

invokeWithFuture及invokeWithCallback 没有关闭,应该是因为HashedWheelTimer的线程是非daemon的导致。这个可以也改一下

@yangl
Copy link
Contributor Author

yangl commented May 22, 2018

嗯 这个加的有

@xmtsui
Copy link
Member

xmtsui commented May 22, 2018

@yangl 整体看了下PR,客户端的问题应该没有了,其他有如下几个问题哈:

  1. RpcServer的线程还没有处理哈。比如RpcServer启动后,RpcClient发起过调用(触发了io线程以及业务线程),此时如果调用RpcServer的stop方法,那么进程不会退出。
  2. 另外,还有一个RemotingServer重复启动,针对启动失败的实例,调用stop方法,无法关闭的bug。(stop方法需要增加判断init状态的情况下,也要触发内部的doStop)
  3. 最后,针对这些bugfix,能都加一下单测么?

@xmtsui xmtsui changed the base branch from master to release/1.4.2 May 22, 2018 12:12
@xmtsui xmtsui added the bug Something isn't working label May 22, 2018
@xmtsui xmtsui added this to the v1.4.2 release milestone May 22, 2018
@xmtsui xmtsui self-requested a review May 22, 2018 12:13
@xmtsui
Copy link
Member

xmtsui commented May 23, 2018

@xmtsui xmtsui merged commit 02da69f into sofastack:release/1.4.2 May 28, 2018
@yangl yangl deleted the bugfix/issues-14 branch May 28, 2018 03:22
@xmtsui
Copy link
Member

xmtsui commented May 30, 2018

commit 0639872 rollback modification for boss group

xmtsui pushed a commit that referenced this pull request Jun 20, 2018
* (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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants