lib/streamaggr: removed unused resetState arg

[AndrewChubatiuk]

Describe Your Changes

resetState is set to true everywhere in a source code except tests. Removing it

Checklist

The following checks are mandatory:

• My change adheres VictoriaMetrics contributing guidelines.

[codecov]

Codecov Report

Attention: Patch coverage is 78.12500% with 14 lines in your changes are missing coverage. Please review.

Project coverage is 57.26%. Comparing base (8aaa828) to head (2a84e90).
Report is 649 commits behind head on master.

❗ Current head 2a84e90 differs from pull request most recent head e2f5cff. Consider uploading reports for the commit e2f5cff to get more accurate results

Files	Patch %	Lines
lib/streamaggr/streamaggr.go	50.00%	9 Missing ⚠️
lib/streamaggr/unique_samples.go	0.00%	3 Missing ⚠️
lib/streamaggr/total.go	60.00%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #6232      +/-   ##
==========================================
- Coverage   60.37%   57.26%   -3.12%     
==========================================
  Files         411      527     +116     
  Lines       76609    72643    -3966     
==========================================
- Hits        46253    41597    -4656     
- Misses      27794    28128     +334     
- Partials     2562     2918     +356     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[hagen1778]

resetState is set to true everywhere in a source code except tests. Removing it

Looks like it is indeed the only purpose - see 6319d02.

Let's keep it, as it will affect benchmark results. I don't know why it is not implemented via global variable, though. It looks like global var set to true in tests could involve less changes.

[AndrewChubatiuk]

interesting
is there any reason to use sync.Map for all aggregators except total?