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

client 端jdk是否存在降级的可能性 #483

Closed
secwang opened this issue Dec 10, 2016 · 15 comments
Closed

client 端jdk是否存在降级的可能性 #483

secwang opened this issue Dec 10, 2016 · 15 comments

Comments

@secwang
Copy link

secwang commented Dec 10, 2016

我只看文档提到client端只依赖guava, guava 是 1.6+ 的,是不是存在可能性,client 端可以使用1.6?

@nobodyiam
Copy link
Member

要支持1.6的话,应该是要改一些代码的。

如String用在switch的地方,泛型的类型推断等。

你可以尝试修改apollo-core/pom.xmlapollo-client/pom.xml中的java.version属性为1.6,然后按照编译提示修改一下。

@secwang
Copy link
Author

secwang commented Dec 10, 2016

我这两天会试一下,issue 暂时不关了

@secwang
Copy link
Author

secwang commented Dec 13, 2016

可以支持 ,需要的可以打我这个 patch https://gist.github.com/secwang/095976ed0808a66c342f154639d2f66b

@secwang secwang closed this as completed Dec 13, 2016
@nobodyiam
Copy link
Member

非常感谢~

@yyf365
Copy link

yyf365 commented Jun 11, 2018

如果现有应用使用的还是1.5的JDK版本,如何能够让client支持呢?

@nobodyiam
Copy link
Member

1.5应该和1.6差不多,可以试试 @secwang 的patch

@DeanJi
Copy link

DeanJi commented Aug 6, 2018

@nobodyiam secwang的patch链接打不开哦。你能发一份给我吗?谢谢。

@nobodyiam
Copy link
Member

@DeanJi 我这里是可以打开的,建议再次尝试一下,或者使用科学上网。

@DeanJi
Copy link

DeanJi commented Aug 7, 2018

@nobodyiam 用了几台电脑试了都不行啊。公司电脑无法科学上网,如果能发一份给我感激不尽啊。
邮箱:[email protected]

@nobodyiam
Copy link
Member

@DeanJi Sent by mail.

@DeanJi
Copy link

DeanJi commented Aug 7, 2018

Thank you. I just saw the patch is also in the Apollo QQ Group.

@Watermelonljf
Copy link

Thank you. I just saw the patch is also in the Apollo QQ Group.

你好能给我也来一分patch吗?公司网出不去

@nobodyiam
Copy link
Member

见附件:0001-1.6.patch.zip

@Watermelonljf
Copy link

见附件:0001-1.6.patch.zip

thank you very much

@zhupeng88
Copy link

请问这是基于哪个版本修改的?

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

No branches or pull requests

6 participants