Uploaded image for project: 'Chukwa'
  1. Chukwa
  2. CHUKWA-37

ChuckwaArchiveBuilder should go away

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 0.1.2, 0.2.0, 0.3.0
    • Fix Version/s: 0.2.0, 0.3.0
    • Component/s: None
    • Labels:
      None
    • Release Note:
      processSinkFiles.sh, buildHourlyArchive.sh and buildDailyArchive.sh have all been removed.

      Description

      org.apache.hadoop.chukwa.extraction.archive.ChuckwaArchiveBuilder exists because of a historical typo, and should go away. Correct class is ChukwaArchiveBuilder [no c].

        Activity

        Hide
        jboulon Jerome Boulon added a comment -

        Raise priority for 0.1.2 release.

        Show
        jboulon Jerome Boulon added a comment - Raise priority for 0.1.2 release.
        Hide
        asrabkin Ari Rabkin added a comment -

        I just fixed this. (I didn't bother with a patch, since we're just deleting a whole file. Is that a violation of our development practices?)

        Show
        asrabkin Ari Rabkin added a comment - I just fixed this. (I didn't bother with a patch, since we're just deleting a whole file. Is that a violation of our development practices?)
        Hide
        asrabkin Ari Rabkin added a comment -

        Turns out that buildDailyArchive.sh and buildHourlyArchive.sh reference this class.

        Are those scripts themselves obsolete? Are they merely stale? Should they be deleted or fixed?

        Show
        asrabkin Ari Rabkin added a comment - Turns out that buildDailyArchive.sh and buildHourlyArchive.sh reference this class. Are those scripts themselves obsolete? Are they merely stale? Should they be deleted or fixed?
        Hide
        asrabkin Ari Rabkin added a comment -

        Deleted processSinkFiles, and buildHourlyArchive/buildDailyArchive, both branch and trunk.

        Show
        asrabkin Ari Rabkin added a comment - Deleted processSinkFiles, and buildHourlyArchive/buildDailyArchive, both branch and trunk.

          People

          • Assignee:
            asrabkin Ari Rabkin
            Reporter:
            asrabkin Ari Rabkin
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development