-
Notifications
You must be signed in to change notification settings - Fork 3
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat: Add key cardinality testing feature (#67)
## Which problem is this PR solving? - We've seen issues where the key cardinality of things being fed to a sampler is longer than the sampler AdjustmentInterval. This gives loadgen the ability to generate fields that will only include all possible values once the generator's sample period has completed. So `mykey=/k50,60` will generate a field called mykey that will have values of cardinality 50, but not all possible values will be used until 60 seconds have passed. Includes testing and benchmark code, as well as a readme update.
- Loading branch information
Showing
4 changed files
with
180 additions
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,66 @@ | ||
package main | ||
|
||
import ( | ||
"fmt" | ||
"strconv" | ||
"strings" | ||
"testing" | ||
"time" | ||
) | ||
|
||
func Test_PeriodicEligibility_checkEligible(t *testing.T) { | ||
words := strings.Split("ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz", "") | ||
notFound := map[string]struct{}{} | ||
for _, w := range words { | ||
notFound[w] = struct{}{} | ||
} | ||
|
||
pe := newPeriodicEligibility(NewRng("hello"), words, 60*time.Second) | ||
t.Run("only some words show up for short period", func(t *testing.T) { | ||
for i := 0; i < 100; i++ { | ||
for p := 0; p < 30; p++ { | ||
word := pe.getEligibleWord(time.Duration(p) * time.Second) | ||
delete(notFound, word) | ||
} | ||
if len(notFound) == 0 { | ||
break | ||
} | ||
} | ||
if len(notFound) != 0 { | ||
t.Errorf("expected some words to be not found, got none") | ||
} | ||
}) | ||
|
||
t.Run("all eligible words show up with full period", func(t *testing.T) { | ||
for i := 0; i < 100; i++ { | ||
for p := 0; p < 60; p++ { | ||
word := pe.getEligibleWord(time.Duration(p) * time.Second) | ||
delete(notFound, word) | ||
} | ||
if len(notFound) == 0 { | ||
break | ||
} | ||
} | ||
if len(notFound) > 0 { | ||
t.Errorf("expected all words to be found, got %v", notFound) | ||
} | ||
}) | ||
} | ||
|
||
func BenchmarkPeriodicEligibility(b *testing.B) { | ||
for _, card := range []int{10, 50, 200} { | ||
var words []string | ||
for i := 0; i < card; i++ { | ||
words = append(words, strconv.Itoa(i)) | ||
} | ||
period := 61 * time.Second | ||
pe := newPeriodicEligibility(NewRng("hello"), words, period) | ||
for p := 0; p < 61; p += 10 { | ||
b.Run(fmt.Sprintf("card_%02d_p_%02d", card, p), func(b *testing.B) { | ||
for i := 0; i < b.N; i++ { | ||
pe.getEligibleWord(time.Duration(p) * time.Second) | ||
} | ||
}) | ||
} | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters