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

重构Web线程池配置文件命名 #1015

Closed
magestacks opened this issue Nov 26, 2022 · 6 comments · Fixed by #1084 or #1087
Closed

重构Web线程池配置文件命名 #1015

magestacks opened this issue Nov 26, 2022 · 6 comments · Fixed by #1084 or #1087
Assignees
Labels
Milestone

Comments

@magestacks
Copy link
Member

当前配置 Web 线程池需要依赖容器名称,如下:

spring:
  dynamic:
    thread-pool:
      # tomcat、undertow、jetty 三种容器线程池,任选其一
      undertow:

考虑到配置不应该随着项目架构而变动,所以修改为:

spring:
  dynamic:
    thread-pool:
      web:
@magestacks magestacks added this to the 1.5.0 milestone Nov 26, 2022
@magestacks magestacks changed the title 修改Web线程池配置文件命名 重构Web线程池配置文件命名 Nov 26, 2022
@magestacks magestacks modified the milestones: 1.5.0, 1.5.1 Nov 26, 2022
@baymax55
Copy link
Contributor

我看了下,除了apollo配置文件使用了,其他示例都没有使用到,只需要后续使用时添加正确即可.无需修改

image

@yanrongzhen
Copy link
Member

认领

@pirme
Copy link
Member

pirme commented Mar 9, 2023

@baymax55 可能没有理解本 issue 的目的,可以参考下 PR 修改内容,谢谢。

@pirme
Copy link
Member

pirme commented Mar 9, 2023

@yanrongzhen 是否可以单独提个 PR 把 Config 模式示例项目的配置更正?

@yanrongzhen
Copy link
Member

@pirme 好的, 没问题

@baymax55
Copy link
Contributor

@baymax55 可能没有理解本 issue 的目的,可以参考下 PR 修改内容,谢谢。

好的,感谢

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment