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

Spring boot autoconfig下能否优化下setMinEvictableIdleTimeMillis与setMaxEvictableIdleTimeMillis顺序问题 #1796

Closed
Plen-wang opened this issue Jun 27, 2017 · 15 comments
Labels
Milestone

Comments

@Plen-wang
Copy link

在springboot下的@ConfigurationProperties自动属性注入的时候,由于spring框架自动注入bean属性数据,导致会优先设置setMaxEvictableIdleTimeMillis,此时还未进行 setMinEvictableIdleTimeMillis属性的设置,但是setMinEvictableIdleTimeMillis的默认值会成为setMaxEvictableIdleTimeMillis判断的值。导致检查失败。

@lihengming
Copy link
Contributor

lihengming commented Jun 28, 2017

Spring Boot 在绑定属性的时候没有按照配置文件内配置项的顺序,貌似是按ASCII码的顺序进行绑定的,建议maxEvictableIdleTimeMillis先自己通过Environment来配置或者不要小于Durid DEFAULT_MIN_EVICTABLE_IDLE_TIME_MILLIS = 1000L * 60L * 30L

@Plen-wang
Copy link
Author

好的,非常感谢。

@lihengming
Copy link
Contributor

@Plen-wang 下个版本会解决这个问题。

@Plen-wang
Copy link
Author

辛苦了。

@wenshao wenshao added this to the 1.1.11 milestone Jul 11, 2017
@wenshao wenshao added the Bug label Jul 11, 2017
@wenshao
Copy link
Member

wenshao commented Jul 13, 2017

https://github.com/alibaba/druid/releases/tag/1.1.1
问题已经修复,请使用新版本

@drtrang
Copy link

drtrang commented Jul 21, 2017

@Plen-wang

问题的原因在这 drtrang/druid-spring-boot#3 (comment)
你应该是用了 application.properties 而不是 application.yml,YAML 格式会按照配置顺序赋值

@Plen-wang
Copy link
Author

@drtrang
嗯嗯,是的。用的是springboot 默认的properties属性文件配置的。

感谢,不知道YAML是根据顺序配置。用YAML是不是注入配置没有那么方便。

@Plen-wang
Copy link
Author

Plen-wang commented Jul 21, 2017

@drtrang
仔细看了下drtrang/druid-spring-boot#3 (comment)
明白原因了,谢谢大咖。

@ichavin
Copy link

ichavin commented Feb 20, 2019

druid-spring-boot-starter 1.1.13版本还存在此问题

@luzhaoren
Copy link

druid-spring-boot-starter 1.1.13版本还存在此问题

确实仍然存在此问题

@lihengming
Copy link
Contributor

@luzhaoren 在解决了,这个问题比较棘手,Spring 读取配置并不是按顺序读的,下个版本修复。

wenshao added a commit that referenced this issue Feb 24, 2019
@wad12302
Copy link

2.0.x 还是存在

@bastYYJ
Copy link

bastYYJ commented Dec 12, 2019

1.1.21还是没修复

@nicolaslonely
Copy link

没有修复 ,如果使用nacos的话 很容易复现

@littlehui
Copy link

1.1.16版本还是如此

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

10 participants