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

Upgrade jetty-server dependency

        Details

        • Type: Improvement
        • Status: Closed
        • Priority: Minor
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: 1.1.0-incubating
        • Component/s: None
        • Labels:
          None

          Issue Links

          depends upon

          Bug - A problem which impairs or prevents the functions of the product. CALCITE-591 Drop support for Java 1.6 (and JDBC 4.0)

          • Major - Major loss of function.
          • Closed
          is related to

          Task - A task that needs to be done. CALCITE-1156 Bump jetty version

          • Major - Major loss of function.
          • Closed

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            julianhyde Julian Hyde added a comment -

            This seems to break the build on JDK 1.6. Can you please take a look? https://travis-ci.org/julianhyde/incubator-calcite/builds/50006360

            Show
            julianhyde Julian Hyde added a comment - This seems to break the build on JDK 1.6. Can you please take a look? https://travis-ci.org/julianhyde/incubator-calcite/builds/50006360
            Hide
            Permalink
            devth Trevor Hartman added a comment -

            Sure, I'll take a look.

            By the way, it might help on the Travis build to set 

            -Dsurefire.useFile=false

            so that we can see errors in the output.

            Show
            devth Trevor Hartman added a comment - Sure, I'll take a look. By the way, it might help on the Travis build to set -Dsurefire.useFile= false so that we can see errors in the output.
            Hide
            Permalink
            devth Trevor Hartman added a comment -

            Jetty 9 only supports Java 7+. Can Calcite drop support for Java 6?

            Show
            devth Trevor Hartman added a comment - Jetty 9 only supports Java 7+. Can Calcite drop support for Java 6?
            Hide
            Permalink
            devth Trevor Hartman added a comment -

            Also, it'd be helpful if Travis reported build status to GitHub PR, e.g. http://blog.travis-ci.com/2012-09-04-pull-requests-just-got-even-more-awesome/.

            Show
            devth Trevor Hartman added a comment - Also, it'd be helpful if Travis reported build status to GitHub PR, e.g. http://blog.travis-ci.com/2012-09-04-pull-requests-just-got-even-more-awesome/ .
            Hide
            Permalink
            julianhyde Julian Hyde added a comment -

            I agree, but I'm not sure that it's possible, given that the PR is against Apache's github mirror https://github.com/apache/incubator-calcite whereas Travis runs against my mirror https://github.com/julianhyde/incubator-calcite.

            Show
            julianhyde Julian Hyde added a comment - I agree, but I'm not sure that it's possible, given that the PR is against Apache's github mirror https://github.com/apache/incubator-calcite whereas Travis runs against my mirror https://github.com/julianhyde/incubator-calcite .
            Hide
            Permalink
            julianhyde Julian Hyde added a comment -

            No objections in principle, but we'd need to reach consensus on the dev list before we do that. Can you start that discussion?

            Show
            julianhyde Julian Hyde added a comment - No objections in principle, but we'd need to reach consensus on the dev list before we do that. Can you start that discussion?
            Hide
            Permalink
            julianhyde Julian Hyde added a comment -

            Fixed in http://git-wip-us.apache.org/repos/asf/incubator-calcite/commit/0cb94951.

            Show
            julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/incubator-calcite/commit/0cb94951 .
            Hide
            Permalink
            julianhyde Julian Hyde added a comment -

            Closing now that 1.1.0-incubating has been released.

            Show
            julianhyde Julian Hyde added a comment - Closing now that 1.1.0-incubating has been released.

              People

              • Assignee:
                julianhyde Julian Hyde
                Reporter:
                devth Trevor Hartman
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development