[internal/comparetest] Make all the exported functions not mutating #17549
Labels
enhancement
New feature or request
good first issue
Good for newcomers
help wanted
Extra attention is needed
internal/comparetest
priority:p2
Medium
Currently, the defensive copying is done before applying Sort only for functions comparing top-level entities like
CompareMetrics
,CompareTraces
, andCompareLogs
. All other exported functions e.g.CompareLogRecordSlices
are mutating because they apply Sort on the provided entities without making a copy before that. This should be fixed. All exportedCompare...
functions must be not mutating.The text was updated successfully, but these errors were encountered: