Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Data Collection
    • Labels:
      None

      Description

      We don't currently deliver anything other than logs reliably. I propose to create a new "wrapping" adaptor, to be named Buffered.
      So you'll be able to say "add Buffered UDPListener 90xy".

      This is going to require some fairly modest refactoring inside ChukwaAgent.processAddCommand, to tease apart the logic for creating Adaptors from the logic for starting them. I think it won't be too bad.

        Activity

        Hide
        Ari Rabkin added a comment -

        Support for RAM-backed buffers. Other buffer management techniques will be easy to implement, given a storage API.

        Show
        Ari Rabkin added a comment - Support for RAM-backed buffers. Other buffer management techniques will be easy to implement, given a storage API.
        Hide
        Ari Rabkin added a comment -

        This patch incorporates a fair bit of cleanup and consolidation of test code. The "generate test file" code that was previously duplicated in several tests has been factored out. But there aren't actually significant functional changes to existing code.

        Since the patch passes unit tests and doesn't involve significant functional changes, I intend to commit this, barring objection.

        Show
        Ari Rabkin added a comment - This patch incorporates a fair bit of cleanup and consolidation of test code. The "generate test file" code that was previously duplicated in several tests has been factored out. But there aren't actually significant functional changes to existing code. Since the patch passes unit tests and doesn't involve significant functional changes, I intend to commit this, barring objection.
        Hide
        Eric Yang added a comment -

        +1 looks good.

        Show
        Eric Yang added a comment - +1 looks good.
        Hide
        Ari Rabkin added a comment -

        I just committed this.

        Show
        Ari Rabkin added a comment - I just committed this.
        Hide
        Hudson added a comment -

        Integrated in Chukwa-trunk #217 (See http://hudson.zones.apache.org/hudson/job/Chukwa-trunk/217/)
        . Support for generalized buffering of adaptor output

        Show
        Hudson added a comment - Integrated in Chukwa-trunk #217 (See http://hudson.zones.apache.org/hudson/job/Chukwa-trunk/217/ ) . Support for generalized buffering of adaptor output

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development