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

[ISSUE-#4631] Remove timer,Use ScheduledThreadPoolExecutor replaced. #4635

Merged
merged 4 commits into from
Jan 8, 2021

Conversation

onewe
Copy link
Collaborator

@onewe onewe commented Jan 6, 2021

Please do not create a Pull Request without creating an issue first.

What is the purpose of the change

fix-#4631

@CLAassistant
Copy link

CLAassistant commented Jan 6, 2021

CLA assistant check
All committers have signed the CLA.

@onewe onewe requested a review from chuntaojun January 7, 2021 07:41
@onewe
Copy link
Collaborator Author

onewe commented Jan 7, 2021

@chuntaojun 大佬,这个CI为啥没过呀??

@KomachiSion KomachiSion added this to the 1.4.1 milestone Jan 7, 2021
@KomachiSion KomachiSion merged commit 79fddc3 into alibaba:develop Jan 8, 2021
@onewe onewe deleted the fix/close_timer_thread branch January 8, 2021 01:49
wjm0729 added a commit to wjm0729/nacos that referenced this pull request Jan 13, 2021
…op-import-v1

* 'develop' of https://github.com/alibaba/nacos:
  naming push service support CSharp client (alibaba#4670)
  remove final
  [ISSUE-alibaba#3880] NamingService Client support pushEmptyProtection. (alibaba#4665)
  [ISSUE-alibaba#4631] Free credential instance when serverHttpAgent shutdown. (alibaba#4634)
  Add CSharp Client
  Just choose one between nacosDomain mode and servers mode.
  upgrade axios version to 0.21.1 (alibaba#4632)
  [ISSUE-alibaba#4631] Remove timer,Use ScheduledThreadPoolExecutor replaced. (alibaba#4635)
  fix: fix jraft response instance (alibaba#4644)
  For alibaba#4291, support remove old raft metadata
  change tokenValidityInSeconds default value
  remove JwtTokenUtils
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants