Concerning the following line of code the initialisation of the slice opentracingFields creates a slice of the size of the number of fields plus one presumable for the log string.
|
opentracingFields := make([]opentracinglog.Field, len(fields)+1) |
However the method the real log fields are added to opentracingFields is done with an append function which causes these additional fields to be added after the nil fields initialized by make. As a result nil/empty fields are passed to span.LogFields() which causes empty log lines to be displayed in the WebUI of Jaeger.
To stop these empty fields being passed the opentracingFields slice should be initialized as zero size, eg: Line 141 above should be;
opentracingFields := make([]opentracinglog.Field, 0)
Concerning the following line of code the initialisation of the slice
opentracingFieldscreates a slice of the size of the number of fields plus one presumable for the log string.go-zap/log/log.go
Line 141 in 6415450
However the method the real log fields are added to
opentracingFieldsis done with anappendfunction which causes these additional fields to be added after the nil fields initialized bymake. As a result nil/empty fields are passed tospan.LogFields()which causes empty log lines to be displayed in the WebUI of Jaeger.To stop these empty fields being passed the
opentracingFieldsslice should be initialized as zero size, eg: Line 141 above should be;