[
https://issues.apache.org/jira/browse/CALCITE-5737?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17728158#comment-17728158
]
Julian Hyde commented on CALCITE-5737:
--------------------------------------
Ah, thanks [~Sergey Nuyanzin]. I saw it a few months back but couldn't find it
just now. Yes that PR seems to fix the problem. Could you or someone else merge
a commit that takes us to JDK 20, and also modifies the release notes.
> 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)
