Bigtop
  1. Bigtop
  2. BIGTOP-51

Layout of Bigtop repository can be optimized

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      he current repo layout is not very self-explanatory. I like
      self-explanatory repo layouts. =) Hence, I'm proposing the following:

      • The current src/pkg contents will go to a new bigtop-packages/src
        directory.
      • The current src/repos directory will move to bigtop-repos - specifically
        to bigtop-repos/apt, since the only file we've got is Debian-specific
        currently.
      • The current top-level POM and src/site directory will move to bigtop-site.
      • test/src/itest-common will move to bigtop-test-framework
      • The code in test/src/integration will get folded in with what's currently
        in test/src/smokes, moved from (module)/src/test to (module)/src/main (since
        these are artifacts containing tests to be run later, not tests to be run at
        build time), and then all moved to bigtop-tests/test-artifacts.
      • test/src/suites will move to bigtop-tests/test-execution
      • docs can either stay as is or be renamed bigtop-docs for consistency.

      http://mail-archives.apache.org/mod_mbox/incubator-bigtop-dev/201108.mbox/%3CCAPbPdOYMquy4gRn7a=LJ+8gzvFRqz2xrxr2sDPG9x9BFvin=GA@mail.gmail.com%3E

      1. BIGTOP-51-mk2.diff.txt
        26 kB
        Andrew Bayer
      2. reorg.txt
        69 kB
        Andrew Bayer
      There are no Sub-Tasks for this issue.

        Activity

        Hide
        Andrew Bayer added a comment -

        So the patch for this is actually too big to attach here - git-svn thinking there are new files when they're actually just moved, etc. The diff is up at http://people.apache.org/~abayer/diffs/BIGTOP-51.diff.

        Show
        Andrew Bayer added a comment - So the patch for this is actually too big to attach here - git-svn thinking there are new files when they're actually just moved, etc. The diff is up at http://people.apache.org/~abayer/diffs/BIGTOP-51.diff .
        Hide
        Roman Shaposhnik added a comment -

        Can you, possibly, attach git status output so that renames are easier to track?

        Show
        Roman Shaposhnik added a comment - Can you, possibly, attach git status output so that renames are easier to track?
        Hide
        Andrew Bayer added a comment -

        Git status output.

        Show
        Andrew Bayer added a comment - Git status output.
        Hide
        Roman Shaposhnik added a comment -

        +1

        Show
        Roman Shaposhnik added a comment - +1
        Hide
        Andrew Bayer added a comment -

        Reorganizing Maven groupIds.

        Show
        Andrew Bayer added a comment - Reorganizing Maven groupIds.
        Hide
        Roman Shaposhnik added a comment -

        After some pondering, here's my take on this: +1 on org.apache.bigtop.itest -> org.apache.bigtop.itest.execution part

        As for org.apache.bigtop.itest -> org.apache.bigtop.itest.artifacts at this point I really thing we should just rename it
        to org.apache.bigtop.test[s].

        Does it make sense?

        Show
        Roman Shaposhnik added a comment - After some pondering, here's my take on this: +1 on org.apache.bigtop.itest -> org.apache.bigtop.itest.execution part As for org.apache.bigtop.itest -> org.apache.bigtop.itest.artifacts at this point I really thing we should just rename it to org.apache.bigtop.test [s] . Does it make sense?
        Hide
        Andrew Bayer added a comment -

        I think this is done enough for 0.2.0 - there's some more that could be done, but I'd prefer to wait til after the release.

        Show
        Andrew Bayer added a comment - I think this is done enough for 0.2.0 - there's some more that could be done, but I'd prefer to wait til after the release.
        Hide
        Roman Shaposhnik added a comment -

        The patch doesn't seem to apply anymore, but it is still an important issue to resolve some day. Hence canceling the patch but leaving the issue open.

        Show
        Roman Shaposhnik added a comment - The patch doesn't seem to apply anymore, but it is still an important issue to resolve some day. Hence canceling the patch but leaving the issue open.

          People

          • Assignee:
            Andrew Bayer
            Reporter:
            Roman Shaposhnik
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Development