Details

    • Type: Improvement Improvement
    • Status: Patch Available
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: v1.2.0, v1.3.0, v1.4.0
    • Fix Version/s: None
    • Component/s: Docs
    • Labels:
      None

      Description

      StressSource is absent at UserGuide, we should fix it

      1. FLUME-1521.patch
        2 kB
        Denny Ye
      2. FLUME-1521-1.patch
        2 kB
        Ashish Paliwal

        Issue Links

          Activity

          Hide
          Brock Noland added a comment -

          Denny, this looks good overall! Thank you for creating the patch to document this! I have just a few nits below:

          1) batchSize should be documented as well

          2) In regards to the description, how about:

          StressSource is allows for stress testing of Flume. Users can configure the body size of each event
          which will have an empty header. The total number and total successful number of events can be
          configured as well.

          3) In regards to maxTotalEvents and maxSuccessfulEvents, how about:

          maxTotalEvents -1 Maximum number of events that will be pushed to Channel
          maxSuccessfulEvents -1 Maximum number of events successful events that will be pushed to Channel

          Show
          Brock Noland added a comment - Denny, this looks good overall! Thank you for creating the patch to document this! I have just a few nits below: 1) batchSize should be documented as well 2) In regards to the description, how about: StressSource is allows for stress testing of Flume. Users can configure the body size of each event which will have an empty header. The total number and total successful number of events can be configured as well. 3) In regards to maxTotalEvents and maxSuccessfulEvents, how about: maxTotalEvents -1 Maximum number of events that will be pushed to Channel maxSuccessfulEvents -1 Maximum number of events successful events that will be pushed to Channel
          Hide
          Mike Percy added a comment -

          Hi Denny,
          Any plans to update this patch based on review feedback?

          Show
          Mike Percy added a comment - Hi Denny, Any plans to update this patch based on review feedback?
          Hide
          Mike Percy added a comment -

          Temporarily cancelling patch available status based on review feedback

          Show
          Mike Percy added a comment - Temporarily cancelling patch available status based on review feedback
          Hide
          Ashish Paliwal added a comment -

          Taking over the JIRA, seems like review comments need to be incorporated

          Show
          Ashish Paliwal added a comment - Taking over the JIRA, seems like review comments need to be incorporated
          Hide
          Ashish Paliwal added a comment -

          Updated the User Guide. We can also update the javadoc, didn't update it as part of the patch so that change context is maintained. Should I raise one more jira or fix it as part of this?

          Show
          Ashish Paliwal added a comment - Updated the User Guide. We can also update the javadoc, didn't update it as part of the patch so that change context is maintained. Should I raise one more jira or fix it as part of this?

            People

            • Assignee:
              Ashish Paliwal
              Reporter:
              Denny Ye
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:

                Development