Flume
  1. Flume
  2. FLUME-1773

File Channel worker thread should not be daemon

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: v1.4.0, v1.3.1
    • Component/s: None
    • Labels:
      None

      Description

      The worker thread should be a normal thread - we should give it a few seconds to stop.

      1. FLUME-1773-3.patch
        5 kB
        Hari Shreedharan
      2. FLUME-1773-2.patch
        5 kB
        Hari Shreedharan
      3. FLUME-1773.patch
        5 kB
        Hari Shreedharan

        Issue Links

          Activity

          Hide
          Hari Shreedharan added a comment -

          There are a couple of things we should change, maybe later. In Log class, this is being leaked out in the constructor, also a thread is being scheduled. Both these will cause issues if we choose to inherit from the Log class later.

          Show
          Hari Shreedharan added a comment - There are a couple of things we should change, maybe later. In Log class, this is being leaked out in the constructor, also a thread is being scheduled. Both these will cause issues if we choose to inherit from the Log class later.
          Hide
          Brock Noland added a comment -

          Hi,

          Yeah I don't see any reason to change that until we want to extend the Log class at which point it will be quite easy to change. Can you link this JIRA to the Review Board Item?

          Brock

          Show
          Brock Noland added a comment - Hi, Yeah I don't see any reason to change that until we want to extend the Log class at which point it will be quite easy to change. Can you link this JIRA to the Review Board Item? Brock
          Hide
          Brock Noland added a comment -

          Also, this patch ignores checkpointInterval which should be used or deprecated.

          Show
          Brock Noland added a comment - Also, this patch ignores checkpointInterval which should be used or deprecated.
          Hide
          Hari Shreedharan added a comment -

          Brock - good catch. The new patch uses the checkpointInterval property to schedule the thread

          Show
          Hari Shreedharan added a comment - Brock - good catch. The new patch uses the checkpointInterval property to schedule the thread
          Hide
          Brock Noland added a comment -

          Thanks for the patch Hari! Committed to trunk and 1.4!

          Show
          Brock Noland added a comment - Thanks for the patch Hari! Committed to trunk and 1.4!
          Hide
          Hudson added a comment -

          Integrated in flume-trunk #338 (See https://builds.apache.org/job/flume-trunk/338/)
          FLUME-1773: File Channel worker thread should not be daemon (Revision e45d2fa33a23aee6d066bd0a4147cb215afa98c4)

          Result = SUCCESS
          brock : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=e45d2fa33a23aee6d066bd0a4147cb215afa98c4
          Files :

          • flume-ng-channels/flume-file-channel/src/main/java/org/apache/flume/channel/file/Log.java
          Show
          Hudson added a comment - Integrated in flume-trunk #338 (See https://builds.apache.org/job/flume-trunk/338/ ) FLUME-1773 : File Channel worker thread should not be daemon (Revision e45d2fa33a23aee6d066bd0a4147cb215afa98c4) Result = SUCCESS brock : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=e45d2fa33a23aee6d066bd0a4147cb215afa98c4 Files : flume-ng-channels/flume-file-channel/src/main/java/org/apache/flume/channel/file/Log.java

            People

            • Assignee:
              Hari Shreedharan
              Reporter:
              Hari Shreedharan
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development