You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Sergey Nuyanzin (Jira)" <ji...@apache.org> on 2023/05/31 22:06:00 UTC
[jira] [Commented] (CALCITE-5737) Support JDK 19, JDK 20
[ https://issues.apache.org/jira/browse/CALCITE-5737?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17728154#comment-17728154 ]
Sergey Nuyanzin commented on CALCITE-5737:
------------------------------------------
just FYI
there is a PR for this https://github.com/apache/calcite/pull/3113
> Support JDK 19, JDK 20
> ----------------------
>
> Key: CALCITE-5737
> URL: https://issues.apache.org/jira/browse/CALCITE-5737
> Project: Calcite
> Issue Type: Bug
> Reporter: Julian Hyde
> Priority: Major
> Fix For: 1.35.0
>
>
> Support JDK 19, JDK 20.
> We currently support JDK (and OpenJDK) versions up to 18. JDK 20 is the latest. We should support it in Calcite 1.35.
> This change would modify history.md (for the upcoming release), add JDK 20 to the GitHub CI, and fix the build. I see a couple of deprecation warnings compiling under JDK 19:
> {noformat}
> core/src/main/java/org/apache/calcite/util/Util.java:1723: warning: [deprecation] Locale(String) in Locale has been deprecated
> return new Locale(strings[0]);
> ^
> core/src/main/java/org/apache/calcite/util/Util.java:1725: warning: [deprecation] Locale(String,String) in Locale has been deprecated
> return new Locale(strings[0], strings[1]);
> ^
> core/src/main/java/org/apache/calcite/util/Util.java:1727: warning: [deprecation] Locale(String,String,String) in Locale has been deprecated
> return new Locale(strings[0], strings[1], strings[2]);
> ^
> error: warnings found and -Werror specified
> core/src/main/java/org/apache/calcite/sql/parser/SqlParserUtil.java:744: warning: [deprecation] Locale(String) in Locale has been deprecated
> locale = new Locale(localeParts[0]);
> ^
> core/src/main/java/org/apache/calcite/sql/parser/SqlParserUtil.java:746: warning: [deprecation] Locale(String,String) in Locale has been deprecated
> locale = new Locale(localeParts[0], localeParts[1]);
> ^
> core/src/main/java/org/apache/calcite/sql/parser/SqlParserUtil.java:748: warning: [deprecation] Locale(String,String,String) in Locale has been deprecated
> locale = new Locale(localeParts[0], localeParts[1], localeParts[2]);
> ^
> {noformat}
> These definitely need to be fixed. I don't know whether there's anything else.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)