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

Separate Avatica from "core" Calcite build

    Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.7.0, avatica-1.7.0
    • Component/s: avatica
    • Labels:
      None

      Description

      Uber-JIRA-issue for tracking the separation of Avatica from the rest of the Calcite build.

      Long term, Avatica can be a standalone project, but we're not there yet in terms of governance (enough people intimately involved). One thing we can do now is to separate Avatica from Calcite and begin "getting them used to" not being next to one another.

      DISCUSS thread: https://mail-archives.apache.org/mod_mbox/calcite-dev/201601.mbox/%3C56ABCCFF.3010205%40gmail.com%3E

        Issue Links

          Activity

          Hide
          elserj Josh Elser added a comment -

          Started working on this tonight and I have an outline up https://github.com/joshelser/calcite/tree/1077-avatica-detach.

          I need to look at it with fresh eyes again, but I don't think it's too bad for a first pass. I'll have to run through a mock release process for Avatica to make sure it's generating reasonable artifacts (I'm not sure yet), checking Javadocs too. The usual.

          There's a bit of duplication in checkstyle configuration files, but I think that's more of an issue with how the "core" build is doing it (https://maven.apache.org/plugins/maven-checkstyle-plugin/examples/multi-module-config.html is what's recommended). That could just be something to change in the future. The checkstyle rules haven't been changing too much.

          Show
          elserj Josh Elser added a comment - Started working on this tonight and I have an outline up https://github.com/joshelser/calcite/tree/1077-avatica-detach . I need to look at it with fresh eyes again, but I don't think it's too bad for a first pass. I'll have to run through a mock release process for Avatica to make sure it's generating reasonable artifacts (I'm not sure yet), checking Javadocs too. The usual. There's a bit of duplication in checkstyle configuration files, but I think that's more of an issue with how the "core" build is doing it ( https://maven.apache.org/plugins/maven-checkstyle-plugin/examples/multi-module-config.html is what's recommended). That could just be something to change in the future. The checkstyle rules haven't been changing too much.
          Hide
          elserj Josh Elser added a comment - - edited

          Julian Hyde, if you have a moment, it would be good to get your input.

          /avatica is no longer built via the top-level calcite pom. I copied the "calcite site" into /avatica and made some modifications (some place holders too until the first release is made). It should "slot" into the Calcite site at https://calcite.apache.org/avatica.

          LMK what you think?

          Show
          elserj Josh Elser added a comment - - edited Julian Hyde , if you have a moment, it would be good to get your input. /avatica is no longer built via the top-level calcite pom. I copied the "calcite site" into /avatica and made some modifications (some place holders too until the first release is made). It should "slot" into the Calcite site at https://calcite.apache.org/avatica . LMK what you think?
          Hide
          julianhyde Julian Hyde added a comment -

          Sounds good.

          Show
          julianhyde Julian Hyde added a comment - Sounds good.
          Hide
          elserj Josh Elser added a comment -

          LMK what you think?

          Sounds good.

          Ok, I'll go ahead and pull this in today then. I've been having some issues getting it to run successfully on Travis (times out in some calcite-core tests). I'll also deploy a current snapshot of the Avatica modules so other devs (hopefully) don't notice the interruption. If we get the release out this week, it shouldn't be too bad.

          I had also stepped through the release process for Avatica (same as the process you have documented) and that worked for me. Hopefully it won't be painful.

          Show
          elserj Josh Elser added a comment - LMK what you think? Sounds good. Ok, I'll go ahead and pull this in today then. I've been having some issues getting it to run successfully on Travis (times out in some calcite-core tests). I'll also deploy a current snapshot of the Avatica modules so other devs (hopefully) don't notice the interruption. If we get the release out this week, it shouldn't be too bad. I had also stepped through the release process for Avatica (same as the process you have documented) and that worked for me. Hopefully it won't be painful.
          Hide
          elserj Josh Elser added a comment -

          Closing due to the completion of all sub-tasks.

          Show
          elserj Josh Elser added a comment - Closing due to the completion of all sub-tasks.
          Hide
          elserj Josh Elser added a comment -

          reopening for new sub-tasks.

          Show
          elserj Josh Elser added a comment - reopening for new sub-tasks.
          Hide
          elserj Josh Elser added a comment -

          Resolved in release Avatica 1.7.0

          Show
          elserj Josh Elser added a comment - Resolved in release Avatica 1.7.0

            People

            • Assignee:
              elserj Josh Elser
              Reporter:
              elserj Josh Elser
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development