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

        Ari Rabkin created issue -
        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.
        Ari Rabkin made changes -
        Field Original Value New Value
        Attachment CHUKWA-395.patch [ 12426506 ]
        Ari Rabkin made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        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.
        Ari Rabkin made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        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
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        61d 11h 53m 1 Ari Rabkin 01/Dec/09 06:05
        Patch Available Patch Available Resolved Resolved
        2d 16h 43m 1 Ari Rabkin 03/Dec/09 22:48

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development