Implements a basic SelectSeries API for querying metrics on profiles #160
Conversation
response.Error = err | ||
return response | ||
} | ||
// todo remove me and add the series to the frame. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@aocenas will take over from here.
} | ||
|
||
// Equal compares two protobuf messages ignoring extra generated proto fields. | ||
func EqualProto(t *testing.T, expected, actual interface{}) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When comparing proto I suggest everyone to use that one, the output is way more useful and it automatically ignore proto fields.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Played a bit around with the parameters and it looked all like it is doing the right thing
Profiles: tt.in, | ||
}), nil) | ||
return q, nil | ||
}, log.NewLogfmtLogger(os.Stdout)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not too sure if this was an oversight or not, feel free to ignore:
}, log.NewLogfmtLogger(os.Stdout)) | |
}, log.NewLogfmtLogger(os.Stderr)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
on my env it doesn't really matter.
Implements a basic SelectSeries API for querying metrics on profiles
No description provided.