Skip to content

Conversation

zzcoding
Copy link

@zzcoding zzcoding commented Sep 6, 2018

修复bug 由于ElasticSearchConnection 中的close方法没有实现,导致在使用druid连接池时,elastic的tcp连接一直增加导致服务器资源占满。

@shi-yuan shi-yuan merged commit 0a333b8 into NLPchina:master Sep 12, 2018
@shi-yuan
Copy link
Member

Thanks!

这么改没毛病,可以解决问题。
不过个人认为,目前的实现机制有点问题,client不应该放在connection里,现在是每次new一个connection就会创建一个client。
其实不然,应该是每次创建一个datasource,才创建一个client

@zzcoding
Copy link
Author

嗯 这个项目还是很不错的呢, 在用的过程中也发现了一些bug,后续有时间会继续fix。

@shi-yuan
Copy link
Member

shi-yuan commented Sep 24, 2018

已将client的创建与销毁放在了datasource里了:5d370f1

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants