Skip to content

Commit

Permalink
Fix some function comments (#4410)
Browse files Browse the repository at this point in the history
Signed-off-by: cui fliter <[email protected]>
Co-authored-by: Yuri Shkuro <[email protected]>
  • Loading branch information
cuishuang and yurishkuro authored May 27, 2023
1 parent 9c787fc commit 9c2d96a
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion cmd/agent/app/reporter/grpc/reporter.go
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ func (r *Reporter) send(ctx context.Context, spans []*model.Span, process *model
return err
}

// addTags appends jaeger tags for the agent to every span it sends to the collector.
// addProcessTags appends jaeger tags for the agent to every span it sends to the collector.
func addProcessTags(spans []*model.Span, process *model.Process, agentTags []model.KeyValue) ([]*model.Span, *model.Process) {
if len(agentTags) == 0 {
return spans, process
Expand Down
2 changes: 1 addition & 1 deletion cmd/query/app/query_parser.go
Original file line number Diff line number Diff line change
Expand Up @@ -326,7 +326,7 @@ func parseBool(r *http.Request, paramName string) (b bool, err error) {
return b, nil
}

// parseSpanKindParam parses the input span kinds to filter for in the metrics query.
// parseSpanKinds parses the input span kinds to filter for in the metrics query.
//
// Valid input span kinds include:
// - "unspecified": when no span kind specified in span.
Expand Down
2 changes: 1 addition & 1 deletion plugin/sampling/strategystore/static/strategy_store.go
Original file line number Diff line number Diff line change
Expand Up @@ -241,7 +241,7 @@ func (h *strategyStore) parseStrategies(strategies *strategies) {
h.storedStrategies.Store(newStore)
}

// mergePerOperationStrategies merges two operation strategies a and b, where a takes precedence over b.
// mergePerOperationSamplingStrategies merges two operation strategies a and b, where a takes precedence over b.
func mergePerOperationSamplingStrategies(
a, b []*api_v2.OperationSamplingStrategy,
) []*api_v2.OperationSamplingStrategy {
Expand Down

0 comments on commit 9c2d96a

Please sign in to comment.