Chukwa
  1. Chukwa
  2. CHUKWA-23

Should really clean up chukwa bin scripts

    Details

    • Type: Task Task
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Duplicate
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Build and Test Code
    • Labels:
      None

      Description

      The chukwa/bin directory is overgrown with bin scripts, not all of which are documented or even still in use. Should really clean this up.

        Activity

        Hide
        Ari Rabkin added a comment -

        Is this issue still live? I will close it as duplicate, barring objection.

        Show
        Ari Rabkin added a comment - Is this issue still live? I will close it as duplicate, barring objection.
        Hide
        Ari Rabkin added a comment -

        I'm taking silence (plus our phone conference on Monday) for consent, and nuking both scripts. This will also close CHUKWA-37.

        Show
        Ari Rabkin added a comment - I'm taking silence (plus our phone conference on Monday) for consent, and nuking both scripts. This will also close CHUKWA-37 .
        Hide
        Ari Rabkin added a comment -

        I'm not sure grouping it into a big script is helpful; I think that would make it harder to understand, not easier.

        However, I'm all for trimming down the dir.
        I propose deleting buildHourlyArchive and buildDailyArchive as well as processSinkFiles. I can do the svn delete if someone votes to approve – no patch needed.

        Show
        Ari Rabkin added a comment - I'm not sure grouping it into a big script is helpful; I think that would make it harder to understand, not easier. However, I'm all for trimming down the dir. I propose deleting buildHourlyArchive and buildDailyArchive as well as processSinkFiles. I can do the svn delete if someone votes to approve – no patch needed.
        Hide
        Eric Yang added a comment -

        The number of script in bin directory is growing out of control. I suggest to change this into a big script like hadoop's bin/hadoop script. This script would look like:

        bin/chukwa help

        chukwa [command] [options]

        • help [command]
        • exec [metrics_type] [start|stop]
        • agent [start|stop]
        • collector [start|stop]
        • demux [start|stop]
        • archive [start|stop] [date range]
        • aggregator [start|stop] [date range]
        • mdl [sequence file]
        • hicc [start|stop]
        Show
        Eric Yang added a comment - The number of script in bin directory is growing out of control. I suggest to change this into a big script like hadoop's bin/hadoop script. This script would look like: bin/chukwa help chukwa [command] [options] help [command] exec [metrics_type] [start|stop] agent [start|stop] collector [start|stop] demux [start|stop] archive [start|stop] [date range] aggregator [start|stop] [date range] mdl [sequence file] hicc [start|stop]
        Hide
        Jerome Boulon added a comment -

        +1

        Show
        Jerome Boulon added a comment - +1
        Hide
        Ari Rabkin added a comment -

        Can we delete processSinkFiles? I believe it has no current users, duplicates existing functionality, and is unmaintained. I would like to delete it.

        Show
        Ari Rabkin added a comment - Can we delete processSinkFiles? I believe it has no current users, duplicates existing functionality, and is unmaintained. I would like to delete it.

          People

          • Assignee:
            Unassigned
            Reporter:
            Ari Rabkin
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development