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

should allow shorter names for adaptors

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Data Collection
    • Labels:
      None

      Description

      Right now, users must specify the complete class name of adaptors. This is less than ideal, since it exposes the Chukwa class structure, and is cumbersome for humans. It would be nice to have some sort of alias mechanism to allow shorter names.

        Activity

        Hide
        asrabkin Ari Rabkin added a comment -

        Somewhere along the way, we started do auto-completion, attempting to prepend o.a.h.c.datacollection.adaptor. This isn't an error condition, and shouldn't generate warn statements in the logs.

        This patch doesn't resolve the issue, but it's helpful.

        Show
        asrabkin Ari Rabkin added a comment - Somewhere along the way, we started do auto-completion, attempting to prepend o.a.h.c.datacollection.adaptor. This isn't an error condition, and shouldn't generate warn statements in the logs. This patch doesn't resolve the issue, but it's helpful.
        Hide
        asrabkin Ari Rabkin added a comment -

        Should clean up log level. Possibly also in 0.2?

        Show
        asrabkin Ari Rabkin added a comment - Should clean up log level. Possibly also in 0.2?
        Hide
        eyang Eric Yang added a comment -

        +1 Looks good.
        +1 for 0.2.

        Show
        eyang Eric Yang added a comment - +1 Looks good. +1 for 0.2.
        Hide
        macyang Mac Yang added a comment -

        This is a nice to have. And since 0.2 is feature complete, I don't think this should go into 0.2.

        Show
        macyang Mac Yang added a comment - This is a nice to have. And since 0.2 is feature complete, I don't think this should go into 0.2.
        Hide
        asrabkin Ari Rabkin added a comment -

        This patch doesn't add a new feature; it just cleans up some misleading logging statements.

        Show
        asrabkin Ari Rabkin added a comment - This patch doesn't add a new feature; it just cleans up some misleading logging statements.
        Hide
        asrabkin Ari Rabkin added a comment -

        Committed to trunk. Holding off for more discussion before either closing issue or committing to 0.2

        Show
        asrabkin Ari Rabkin added a comment - Committed to trunk. Holding off for more discussion before either closing issue or committing to 0.2
        Hide
        eyang Eric Yang added a comment -

        I am fine with this in trunk only.

        Show
        eyang Eric Yang added a comment - I am fine with this in trunk only.
        Hide
        asrabkin Ari Rabkin added a comment -

        Marking issue as still open, since patch didn't really address underlying issue, just clarified status quo.

        Show
        asrabkin Ari Rabkin added a comment - Marking issue as still open, since patch didn't really address underlying issue, just clarified status quo.
        Hide
        hudson Hudson added a comment -
        Show
        hudson Hudson added a comment - Integrated in Chukwa-trunk #61 (See http://hudson.zones.apache.org/hudson/job/Chukwa-trunk/61/ )
        Hide
        asrabkin Ari Rabkin added a comment -

        This probably isn't going to be changed for 0.3.

        Show
        asrabkin Ari Rabkin added a comment - This probably isn't going to be changed for 0.3.

          People

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

            Dates

            • Created:
              Updated:

              Development