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

这个项目的依赖可以进一步简化 #58

Closed
xusheng1987 opened this issue Oct 8, 2016 · 5 comments
Closed

这个项目的依赖可以进一步简化 #58

xusheng1987 opened this issue Oct 8, 2016 · 5 comments

Comments

@xusheng1987
Copy link

fluent-hc和jodd-http我看了下项目里没用到,可以去除,guava仅在一个地方用到了Lists.newArrayList,此处可以用jdk的List代替,至于joor,不知道能不能用jdk自身的反射代替

@binarywang
Copy link
Member

感谢仔细分析,确实有些属于历史遗留问题,有些属于个人爱好加入的,我们再仔细排查一下,去掉一些不必要的jar包依赖

binarywang added a commit that referenced this issue Oct 10, 2016
@belerweb
Copy link

我也发现了,支持去掉

@binarywang
Copy link
Member

guava 作为一个比较有用且常用的类库,以后代码重构可能会用到比较多,暂时保留。

binarywang added a commit that referenced this issue Oct 27, 2016
@jobmission
Copy link

guava 依赖较多,可否再优化,包括jedis依赖,无法排除掉。
@binarywang

@binarywang
Copy link
Member

guava 依赖较多,可否再优化,包括jedis依赖,无法排除掉。 @binarywang

欢迎自己提供好的优化方案,直接PR即可

@Wechat-Group Wechat-Group locked as too heated and limited conversation to collaborators Feb 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants