Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 0.5.0, 0.6.0
    • Fix Version/s: 0.6.0
    • Component/s: Data Collection
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Occassionally when chukwa agent is restarted, it complains about the configured http port already in use. The TCP connection will show as TIME_WAIT state. The solution is to use the socket reuse TCP option.

      1. CHUKWA-710.patch4
        5 kB
        shreyas subramanya
      2. CHUKWA-710.patch3
        5 kB
        shreyas subramanya
      3. CHUKWA-710.patch2
        2 kB
        shreyas subramanya
      4. chukwa_710.patch
        1 kB
        shreyas subramanya

        Activity

        Hide
        eyang Eric Yang added a comment -

        +1

        Show
        eyang Eric Yang added a comment - +1
        Hide
        shreyass123 shreyas subramanya added a comment -

        Thanks, committed this.

        Show
        shreyass123 shreyas subramanya added a comment - Thanks, committed this.
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Chukwa-trunk #481 (See https://builds.apache.org/job/Chukwa-trunk/481/)
        CHUKWA-710. Set TCP socket reuse option (shreyas: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1589550)

        • /chukwa/trunk/src/main/java/org/apache/hadoop/chukwa/datacollection/adaptor/SocketAdaptor.java
        • /chukwa/trunk/src/main/java/org/apache/hadoop/chukwa/datacollection/agent/ChukwaAgent.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Chukwa-trunk #481 (See https://builds.apache.org/job/Chukwa-trunk/481/ ) CHUKWA-710 . Set TCP socket reuse option (shreyas: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1589550 ) /chukwa/trunk/src/main/java/org/apache/hadoop/chukwa/datacollection/adaptor/SocketAdaptor.java /chukwa/trunk/src/main/java/org/apache/hadoop/chukwa/datacollection/agent/ChukwaAgent.java
        Hide
        eyang Eric Yang added a comment -

        Please update the CHANGES.txt to reflect this change.

        Show
        eyang Eric Yang added a comment - Please update the CHANGES.txt to reflect this change.
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Chukwa-trunk #482 (See https://builds.apache.org/job/Chukwa-trunk/482/)
        CHUKWA-710. Added contributor name (shreyas: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1589584)

        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Chukwa-trunk #482 (See https://builds.apache.org/job/Chukwa-trunk/482/ ) CHUKWA-710 . Added contributor name (shreyas: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1589584 ) /chukwa/trunk/CHANGES.txt CHUKWA-710 . Corrected the category of this item in CHANGES.txt (shreyas: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1589583 ) /chukwa/trunk/CHANGES.txt CHUKWA-710 . updated CHANGES.txt (shreyas: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1589581 ) /chukwa/trunk/CHANGES.txt
        Hide
        shreyass123 shreyas subramanya added a comment -

        Re-submitting patch after adding retry mechanism.

        Show
        shreyass123 shreyas subramanya added a comment - Re-submitting patch after adding retry mechanism.
        Hide
        shreyass123 shreyas subramanya added a comment -

        Added testcase

        Show
        shreyass123 shreyas subramanya added a comment - Added testcase
        Hide
        eyang Eric Yang added a comment -

        Could we follow the 2 space code indentation? Thanks

        Show
        eyang Eric Yang added a comment - Could we follow the 2 space code indentation? Thanks
        Hide
        shreyass123 shreyas subramanya added a comment -

        Corrected indentation.

        Show
        shreyass123 shreyas subramanya added a comment - Corrected indentation.
        Hide
        eyang Eric Yang added a comment -

        +1

        Show
        eyang Eric Yang added a comment - +1
        Hide
        shreyass123 shreyas subramanya added a comment -

        Commited the latest patch.

        Show
        shreyass123 shreyas subramanya added a comment - Commited the latest patch.
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Chukwa-trunk #493 (See https://builds.apache.org/job/Chukwa-trunk/493/)
        CHUKWA-710. Set TCP socket reuse option for server sockets (shreyas: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1606952)

        • /chukwa/trunk/CHANGES.txt
        • /chukwa/trunk/src/main/java/org/apache/hadoop/chukwa/datacollection/adaptor/SocketAdaptor.java
        • /chukwa/trunk/src/test/java/org/apache/hadoop/chukwa/datacollection/adaptor/TestSocketAdaptor.java
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Chukwa-trunk #493 (See https://builds.apache.org/job/Chukwa-trunk/493/ ) CHUKWA-710 . Set TCP socket reuse option for server sockets (shreyas: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1606952 ) /chukwa/trunk/CHANGES.txt /chukwa/trunk/src/main/java/org/apache/hadoop/chukwa/datacollection/adaptor/SocketAdaptor.java /chukwa/trunk/src/test/java/org/apache/hadoop/chukwa/datacollection/adaptor/TestSocketAdaptor.java
        Hide
        shreyass123 shreyas subramanya added a comment -

        Fixed logic error in TestBackpressure testcase.

        Show
        shreyass123 shreyas subramanya added a comment - Fixed logic error in TestBackpressure testcase.
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Chukwa-trunk #494 (See https://builds.apache.org/job/Chukwa-trunk/494/)
        CHUKWA-710 Fix TestBackpressure testcase (shreyas: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1607186)

        • /chukwa/trunk/src/test/java/org/apache/hadoop/chukwa/datacollection/collector/TestBackpressure.java
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Chukwa-trunk #494 (See https://builds.apache.org/job/Chukwa-trunk/494/ ) CHUKWA-710 Fix TestBackpressure testcase (shreyas: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1607186 ) /chukwa/trunk/src/test/java/org/apache/hadoop/chukwa/datacollection/collector/TestBackpressure.java

          People

          • Assignee:
            shreyass123 shreyas subramanya
            Reporter:
            shreyass123 shreyas subramanya
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development