[CALCITE-5746] Support JDK 19

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.35.0
• Labels: pull-request-available
• Assignee: Sergey Nuyanzin
• Reporter: Julian Hyde
• Created: 02/Jun/23 19:12
• Updated: 27/Jul/23 10:22
• Resolved: 03/Jun/23 12:22

Description

Support JDK 19 in Calcite. Modify history.md and remove calls to deprecated Locale constructors. To be fixed before Calcite 1.35.

The following deprecation warnings currently occur when compiling under JDK 19:

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]);
               ^

Issue Links

• relates to: CALCITE-5737 Support JDK 21, JDK22
• relates to: CALCITE-5567 Failed to apply plugin 'de.thetaphi.forbiddenapis' when running ./gradlew build w/ java19
• links to: GitHub Pull Request #3113
• links to: GitHub Pull Request #3113