Skip to content

Commit

Permalink
cloud/v1: Aggregation in a dedicated pkg (#3063)
Browse files Browse the repository at this point in the history
It moves in a dedicated v1 package, the current cloud output's aggregating and flushing samples operations. 
It mostly split the test life cycle management from the data management. It simplifies the next PR where we will introduce new aggregating and flushing methods.
  • Loading branch information
codebien authored May 16, 2023
1 parent 9aa9ede commit 311d94f
Show file tree
Hide file tree
Showing 9 changed files with 1,297 additions and 1,170 deletions.
555 changes: 90 additions & 465 deletions output/cloud/output.go

Large diffs are not rendered by default.

709 changes: 6 additions & 703 deletions output/cloud/output_test.go

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions output/cloud/bench_test.go → output/cloud/v1/bench_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ import (
)

func BenchmarkAggregateHTTP(b *testing.B) {
out, err := newOutput(output.Params{
out, err := newTestOutput(output.Params{
Logger: testutils.NewLogger(b),
JSONConfig: json.RawMessage(`{"noCompress": true, "aggregationCalcInterval": "200ms","aggregationPeriod": "200ms"}`),
ScriptOptions: lib.Options{
Expand Down Expand Up @@ -292,7 +292,7 @@ func BenchmarkHTTPPush(b *testing.B) {
},
)

out, err := newOutput(output.Params{
out, err := newTestOutput(output.Params{
Logger: testutils.NewLogger(b),
JSONConfig: json.RawMessage(fmt.Sprintf(`{
"host": "%s",
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
Loading

0 comments on commit 311d94f

Please sign in to comment.