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

Discuss the time to increase the minimum JDK compilation version of the ShardingSphere project to 11 #20935

Closed
linghengqian opened this issue Sep 12, 2022 · 14 comments

Comments

@linghengqian
Copy link
Member

linghengqian commented Sep 12, 2022

Feature Request

For English only, other languages will not accept.

Please pay attention on issues you submitted, because we maybe need more details.
If no response anymore and we cannot make decision by current information, we will close it.

Please answer these questions before submitting your issue. Thanks!

Is your feature request related to a problem?

Describe the feature you would like.

  • Starting from Antlr 4.10, it requires users to have a minimum JDK version of 11. Refer to https://github.com/antlr/antlr4/releases/tag/4.10.1 .

  • And for Hibernate which relies on Antlr, it uses Antlr 4.10 from Hibernate 6.0.1, Hibernate 6.1.x works on SpringBoot OSS 3.0.0-M4, for ShardingSphere JDBC 5.2.0 which uses Antlr 4.9.2, this breaks ShardingSphere JDBC Use of hybrid deployment form with zookeeper/etcd.

  • I think it's time to consider the point in time when the minimum compiled version of ShardingSphere is raised to JDK11. I think this event should appear later than November 2022, when SpringBoot OSS 3.0.0 releases the first Release version. For the master branch of ShardingSphere, in which version should it raise the minimum build version? Is it ShardingSphere 5.3.0 or ShardingSphere 6.0.0?

  • Refer to https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-3.0.0-M4-Release-Notes#hibernate-61 .

@github-actions
Copy link

github-actions bot commented Oct 8, 2022

Hello , this issue has not received a reply for several days.
This issue is supposed to be closed.

@github-actions github-actions bot closed this as completed Oct 8, 2022
@linghengqian
Copy link
Member Author

  • I'm reopening this issue because no one has responded to the time point, and I'll submit a PR directly if it's ok.

@TeslaCN
Copy link
Member

TeslaCN commented Oct 11, 2022

  • I'm reopening this issue because no one has responded to the time point, and I'll submit a PR directly if it's ok.

This may impact a lot of users. It's better to discuss breaking changes in mailing list.

@linghengqian
Copy link
Member Author

linghengqian commented Oct 11, 2022

@linghengqian linghengqian changed the title The JDK minimum compiled version of the ShardingSphere project bump the timeline of JDK11 Discuss the time to increase the minimum JDK compilation version of the ShardingSphere project to 11 Dec 28, 2022
@zhfeng
Copy link
Contributor

zhfeng commented Jan 11, 2023

Vote on github issue? Is it an official process? I think it should happen on the mailing list although it seems few peope take care of it.

@linghengqian
Copy link
Member Author

  • @zhfeng The official voting procedure should be transferred to the maillist, this issue should not be responsible for any related voting from the process, and I am not a Committer.

  • The reason I didn't turn on the maillist was simply because it wasn't my need, which was @shanrenxj's need in the first place.

@zhfeng
Copy link
Contributor

zhfeng commented Jan 11, 2023

@linghengqian I'm not a Committer either. But it is free to ask a voting on the community, the PMC has a binding +1 and anyone can show his/her opinion.

@zhfeng
Copy link
Contributor

zhfeng commented Jan 11, 2023

Hi @terrymanu @linghengqian @TeslaCN @taojintianxia @shanrenxj,

I raise a thread Plan to drop JDK 8 on the mailing list, and please review and discuss there. I'm open to any idea.

Thanks!

@shanrenxj
Copy link

It's time to give those programmers who refuse to embrace new things a head start. Let them stay in the old shardingsphere.

@zhfeng
Copy link
Contributor

zhfeng commented Jan 12, 2023

Hi @shanrenxj,

We are not pushing people but trying to figure it out. Currently I'm working with @linghengqian to resolve the antlr4 upgrading issue by not breaking the JDK 8 support.

The most thing is like

  • We need to change to build shardingsphere with JDK >= 11 only.
  • But it is still compatitable with JDK 1.8 which the people should be good at runtime.

@linghengqian
Copy link
Member Author

@zhfeng
Copy link
Contributor

zhfeng commented Jan 12, 2023

No, it should be OK without this skip propery with antlr4. I'm trying to prepare a PR to force build shardingsphere with JDK 11. @linghengqian please go head with your current works on GraalVM. I'm looking forward the shardingsphere-native-proxy which will be awsome.

@zhfeng
Copy link
Contributor

zhfeng commented Jan 13, 2023

@linghengqian Please review #23550

@linghengqian
Copy link
Member Author

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

Successfully merging a pull request may close this issue.

7 participants