[Asterisk-code-review] StatsD: Add sample rate compatibility (asterisk[master])

[Kevin Harwell]

Kevin Harwell has posted comments on this change.

Change subject: StatsD: Add sample rate compatibility
......................................................................


Patch Set 1: Code-Review-1

(2 comments)

https://gerrit.asterisk.org/#/c/1571/1/apps/app_statsd.c
File apps/app_statsd.c:

Line 64: 				greater than or equal to 1 will always be sent, and any rate
       : 				between 1 and 0 will be left up to chance.</para>
This sounds a bit odd. What does "left up to chance" mean?


Line 237: 	if (!isdigit(*sample_rate)) {
Is it suppose to be possible for the user to enter the sample rate as a double (seems like it from the help description and the later conversion)? If so this only validates integers.


-- 
To view, visit https://gerrit.asterisk.org/1571
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I11d315d0a5034fffeae1178e650aa8264485ed52
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Tyler Cambron <tcambron at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-HasComments: Yes