Details

    • Type: New Feature New Feature
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Data Collection
    • Labels:
      None
    • Release Note:
      Adaptor for UDP sockets.

      Description

      We should have an adaptor for UDP packets.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        8h 10m 1 Ari Rabkin 17/Dec/09 12:48
        Patch Available Patch Available Resolved Resolved
        1d 7h 31m 1 Ari Rabkin 18/Dec/09 20:19
        Hide
        Hudson added a comment -
        Show
        Hudson added a comment - Integrated in Chukwa-trunk #235 (See http://hudson.zones.apache.org/hudson/job/Chukwa-trunk/235/ )
        Hide
        Ari Rabkin added a comment -

        I accidentally had the line:

        import org.apache.hadoop.chukwa.datacollection.adaptor.filetailer.RCheckFTAdaptor;
        in the test case for this patch. Gonna see if Hudson resolves itself now that CHUKWA-421 has been committed.

        Oops.

        Show
        Ari Rabkin added a comment - I accidentally had the line: import org.apache.hadoop.chukwa.datacollection.adaptor.filetailer.RCheckFTAdaptor; in the test case for this patch. Gonna see if Hudson resolves itself now that CHUKWA-421 has been committed. Oops.
        Hide
        Eric Yang added a comment -

        This patch seems to have some problem, see:

        http://hudson.zones.apache.org/hudson/job/Chukwa-trunk/232/

        Show
        Eric Yang added a comment - This patch seems to have some problem, see: http://hudson.zones.apache.org/hudson/job/Chukwa-trunk/232/
        Ari Rabkin made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Ari Rabkin added a comment -

        I just committed this. Rationale for commit before review: all new code, doesn't change any existing features so no impact on other users.

        Show
        Ari Rabkin added a comment - I just committed this. Rationale for commit before review: all new code, doesn't change any existing features so no impact on other users.
        Ari Rabkin made changes -
        Attachment CHUKWA-431.patch [ 12428297 ]
        Hide
        Ari Rabkin added a comment -

        Now with unit test.

        I intend to commit this tomorrow or so, barring objection.

        Show
        Ari Rabkin added a comment - Now with unit test. I intend to commit this tomorrow or so, barring objection.
        Ari Rabkin made changes -
        Attachment CHUKWA-431.patch [ 12428263 ]
        Ari Rabkin made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Ari Rabkin made changes -
        Field Original Value New Value
        Attachment CHUKWA-431.patch [ 12428263 ]
        Hide
        Ari Rabkin added a comment -

        Not yet ready to commit. Needs tests. But showing what I have.

        Show
        Ari Rabkin added a comment - Not yet ready to commit. Needs tests. But showing what I have.
        Ari Rabkin created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development