Details
-
Type:
Improvement
-
Status: Patch Available
-
Priority:
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
Issue Links
- links to
Activity
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?
Temporarily cancelling patch available status based on review feedback
Show
Mike Percy
added a comment - Temporarily cancelling patch available status based on review feedback
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
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?
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