On Tue, 15 Jun 2021 18:01:18 +0900 Michael Paquier <mich...@paquier.xyz> wrote:
> On Tue, Jun 15, 2021 at 05:15:14PM +0900, Yugo NAGATA wrote: > > On Tue, 15 Jun 2021 10:05:29 +0200 (CEST) Fabien COELHO > > <coe...@cri.ensmp.fr> wrote: > > It was not a problem because --sampling-rate --aggregate-interval cannot be > > used at the same time. > > Yep, you are right, thanks. I have missed that both options cannot be > specified at the same time. Maybe, adding Assert(sample_rate == 0.0 || agg_interval == 0) or moving the check of sample_rate into the else block could improve code readability? -- Yugo NAGATA <nag...@sraoss.co.jp>
