You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shenyu.apache.org by wy...@sina.com on 2021/05/17 07:44:58 UTC

回复:Re: Re: NacosClientRegisterRepository close need add Service shutdown?

This problem has fix and merged to master in this pr.https://github.com/dromara/shenyu/pull/1472----- 原始邮件 -----
发件人:Yu Xiao <xi...@apache.org>
收件人:dev@shenyu.apache.org
主题:Re: Re: NacosClientRegisterRepository close need add Service shutdown?
日期:2021年05月17日 14点11分

Yes,nedd to fix
<wy...@sina.com> 于2021年5月15日周六 下午1:15写道:
> I think so. I will make a pr to fix it.
> ----- 原始邮件 -----
> 发件人:wei liu <lw...@apache.org>
> 收件人:dev@shenyu.apache.org
> 主题:Re: NacosClientRegisterRepository close need add Service shutdown?
> 日期:2021年05月15日 13点09分
>
> If they have a close method, can add.
> <wy...@sina.com> 于2021年5月15日周六 下午12:34写道:
> > In the development of nacos test cases. Recently discovered
> > NacosClientRegisterRepository method is empty. Whether CLOSE needs to be
> > shutdown ConfigService and NameService?
>

Re: Re: Re: NacosClientRegisterRepository close need add Service shutdown?

Posted by Yu Xiao <xi...@apache.org>.
Hi

thanks  fix  it,  committers wil review ....

<wy...@sina.com> 于2021年5月17日周一 下午3:45写道:

> This problem has fix and merged to master in this pr.
> https://github.com/dromara/shenyu/pull/1472----- 原始邮件 -----
> 发件人:Yu Xiao <xi...@apache.org>
> 收件人:dev@shenyu.apache.org
> 主题:Re: Re: NacosClientRegisterRepository close need add Service shutdown?
> 日期:2021年05月17日 14点11分
>
> Yes,nedd to fix
> <wy...@sina.com> 于2021年5月15日周六 下午1:15写道:
> > I think so. I will make a pr to fix it.
> > ----- 原始邮件 -----
> > 发件人:wei liu <lw...@apache.org>
> > 收件人:dev@shenyu.apache.org
> > 主题:Re: NacosClientRegisterRepository close need add Service shutdown?
> > 日期:2021年05月15日 13点09分
> >
> > If they have a close method, can add.
> > <wy...@sina.com> 于2021年5月15日周六 下午12:34写道:
> > > In the development of nacos test cases. Recently discovered
> > > NacosClientRegisterRepository method is empty. Whether CLOSE needs to
> be
> > > shutdown ConfigService and NameService?
> >
>