Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-5746

Support JDK 19

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.35.0
    • None

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

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Sergey Nuyanzin Sergey Nuyanzin
            julianhyde Julian Hyde
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment