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

        Activity

        Hide
        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
        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
        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
        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
        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
        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?
        Show
        julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/incubator-calcite/commit/0cb94951 .
        Hide
        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