From: "Toke Høiland-Jørgensen" <notifications@github.com> To: tohojo/flent <flent@noreply.github.com> Cc: Subscribed <subscribed@noreply.github.com> Subject: Re: [Flent-users] [tohojo/flent] packet loss stats (#106) Date: Mon, 16 Oct 2017 15:54:03 +0000 (UTC) [thread overview] Message-ID: <tohojo/flent/issues/106/336931544@github.com> (raw) In-Reply-To: <tohojo/flent/issues/106@github.com> [-- Attachment #1.1: Type: text/plain, Size: 2881 bytes --] Pete Heist <notifications@github.com> writes: > Thanks! I made most of your changes (-o was particularly broken, so > this is a better solution), except: Cool! > - I'm still thinking about whether to default durations to seconds or > not. I'm using Go's default duration flag parsing, and I like the > explicitness of seeing the units. Hmm, I see your point about explicit being better, but again it's surprising. A compromise could be to detect it and warn the user explicitly (i.e., instead of just printing out the whole help message you could go "you specified a unitless duration. did you mean Xs?"). > - I like that the JSON is written even after ctrl-C, so that > interrupting a long test doesn't mean you lose all your results. But > maybe if you're sending output to stdout, it's not a good idea (or > even breaks some convention?) This is a trade-off between not accidentally dropping data and not having to clean up after a tool that you clearly just interrupted. I would definitely say don't output the JSON to STDOUT when interrupted (but the stats are fine). Writing to file, hmm, I dunno... > I changed it to ping-like behavior (although there is now -q for no > per-packet results and -qq for no output at all). But just to explain > the thought process, I felt that the default of five round trips in > one second, which produces a reasonable approximation of all relevant > stats in a short period of time, was better than waiting in > anticipation for the next one second ping. In one second you could > already be reviewing stats. :) Also, since I think IRTT will be > typically used for lower intervals than ping, not defaulting to > per-packet output made sense to me. I don't need to do things just > because of tradition. **However**, I took your advice because we're > all so accustomed to ping for so many years now, that what I like as a > default might be uncomfortable or annoying to others, and I don't wish > for people to get that feeling. Right, I see your point. But yeah, I think having it be an alternative to ping for interactive use is a valid use case, and for that mirroring what people are used to is probably better. Besides, if someone wants to measure things more accurately, they are probably going to be fiddling with the options anyways... > Also, If there's a reason I should do my tests with iperf2 instead, > I'm all ears, as I'm a "scientist," not attached to my own work. :) I haven't tried out the iperf code yet, but from the discussion on the list yours has several features that iperf is lacking, most notably the handshake (but also way more detailed statistics). So I definitely don't think your efforts are wasted :) -Toke -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/tohojo/flent/issues/106#issuecomment-336931544 [-- Attachment #1.2: Type: text/html, Size: 7720 bytes --] [-- Attachment #2: Type: text/plain, Size: 151 bytes --] _______________________________________________ Flent-users mailing list Flent-users@flent.org http://flent.org/mailman/listinfo/flent-users_flent.org
next prev parent reply other threads:[~2017-10-16 15:54 UTC|newest] Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top [not found] <tohojo/flent/issues/106@github.com> 2017-09-26 9:41 ` Toke Høiland-Jørgensen 2017-09-26 18:35 ` Toke Høiland-Jørgensen 2017-10-10 9:08 ` Toke Høiland-Jørgensen 2017-10-15 17:18 ` Toke Høiland-Jørgensen 2017-10-16 15:54 ` Toke Høiland-Jørgensen [this message] 2017-10-31 11:12 ` Toke Høiland-Jørgensen 2017-10-31 12:02 ` Toke Høiland-Jørgensen 2017-11-16 9:57 ` Toke Høiland-Jørgensen 2017-11-16 12:15 ` Toke Høiland-Jørgensen 2017-11-16 12:17 ` Toke Høiland-Jørgensen 2017-11-16 12:48 ` Toke Høiland-Jørgensen 2017-11-16 13:07 ` Toke Høiland-Jørgensen 2017-11-16 13:18 ` Toke Høiland-Jørgensen 2017-11-16 14:47 ` Toke Høiland-Jørgensen 2017-11-16 18:09 ` Toke Høiland-Jørgensen 2017-11-16 18:11 ` Toke Høiland-Jørgensen 2017-11-17 14:58 ` Toke Høiland-Jørgensen 2017-11-18 13:04 ` Toke Høiland-Jørgensen [not found] ` <tohojo/flent/issues/106/345442147@github.com> 2017-11-18 19:07 ` Dave Taht 2017-11-20 12:11 ` Toke Høiland-Jørgensen 2017-11-20 13:21 ` Toke Høiland-Jørgensen 2017-11-20 21:14 ` Dave Taht 2017-11-20 21:44 ` Dave Taht 2017-11-20 16:01 ` Toke Høiland-Jørgensen 2017-11-20 20:58 ` Toke Høiland-Jørgensen 2017-11-21 10:36 ` Toke Høiland-Jørgensen 2017-11-21 14:53 ` Toke Høiland-Jørgensen 2017-11-21 21:56 ` Toke Høiland-Jørgensen 2017-11-22 7:49 ` Toke Høiland-Jørgensen 2017-11-22 12:03 ` Toke Høiland-Jørgensen 2017-11-22 12:05 ` Toke Høiland-Jørgensen 2017-11-22 12:06 ` Toke Høiland-Jørgensen 2017-11-22 13:34 ` Toke Høiland-Jørgensen 2017-11-22 21:57 ` Toke Høiland-Jørgensen [not found] ` <tohojo/flent/issues/106/332043470@github.com> 2017-11-26 16:49 ` Matthias Tafelmeier
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style List information: https://lists.flent.org/postorius/lists/flent-users.flent.org/ * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=tohojo/flent/issues/106/336931544@github.com \ --to=notifications@github.com \ --cc=flent@noreply.github.com \ --cc=reply+01ead4bf64ca07f195418336e8fdcfa1bbcb46abdd1ecf6e92cf0000000115fc981b92a169ce0ceefa50@reply.github.com \ --cc=subscribed@noreply.github.com \ --subject='Re: [Flent-users] [tohojo/flent] packet loss stats (#106)' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox