Skip to content

Commit

Permalink
fixup! Fix engine test by using query hash instead of serialized params
Browse files Browse the repository at this point in the history
Signed-off-by: Christian Haudum <[email protected]>
  • Loading branch information
chaudum committed Jul 9, 2024
1 parent b7cb579 commit c778d52
Showing 1 changed file with 5 additions and 7 deletions.
12 changes: 5 additions & 7 deletions pkg/logql/engine_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -2738,14 +2738,12 @@ func paramsID(p interface{}) string {
case SelectLogParams:
case SelectSampleParams:
return fmt.Sprintf("%d", params.Plan.Hash())
default:
b, err := json.Marshal(p)
if err != nil {
panic(err)
}
return strings.ReplaceAll(string(b), " ", "")
}
panic("invalid params type")
b, err := json.Marshal(p)
if err != nil {
panic(err)
}
return strings.ReplaceAll(string(b), " ", "")
}

type logData struct {
Expand Down

0 comments on commit c778d52

Please sign in to comment.