Skip to content

Commit

Permalink
doc(das): doc the sampling window param
Browse files Browse the repository at this point in the history
  • Loading branch information
renaynay committed Dec 4, 2023
1 parent 6ff0b15 commit 3a56040
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions das/options.go
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,9 @@ type Parameters struct {
// ConcurrencyLimit.
SampleTimeout time.Duration

// TODO @renaynay: DOC
// SamplingWindow determines the time window that headers should fall into
// in order to be sampled. If set to 0, the sampling window will include
// all headers.
SamplingWindow time.Duration
}

Expand Down Expand Up @@ -160,8 +162,7 @@ func WithSampleTimeout(sampleTimeout time.Duration) Option {
}

// WithSamplingWindow is a functional option to configure the DASer's
// `SamplingWindow` parameter which determines the time window that
// headers should fall into in order to be sampled.
// `SamplingWindow` parameter.
func WithSamplingWindow(samplingWindow time.Duration) Option {
return func(d *DASer) {
d.params.SamplingWindow = samplingWindow
Expand Down

0 comments on commit 3a56040

Please sign in to comment.