Skip to content

Commit

Permalink
Applied requested changes
Browse files Browse the repository at this point in the history
  • Loading branch information
willianpc committed Mar 21, 2022
1 parent 594741c commit 65938a1
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 4 deletions.
2 changes: 1 addition & 1 deletion json_span.go
Original file line number Diff line number Diff line change
Expand Up @@ -307,7 +307,7 @@ func readArrayStringTag(dst *[]string, tag interface{}) {
case []string:
*dst = s
default:
*dst = []string{}
*dst = nil
}
}

Expand Down
10 changes: 7 additions & 3 deletions registered_span.go
Original file line number Diff line number Diff line change
Expand Up @@ -1374,10 +1374,14 @@ func newPostgreSQLSpanTags(span *spanS) postgreSQLSpanTags {

// RedisSpanTags contains fields within the `data.redis` section of an OT span document
type RedisSpanTags struct {
Connection string `json:"connection"`
Command string `json:"command,omitempty"`
// Connection is the host and port where the Redis server is running
Connection string `json:"connection"`
// Command is the Redis command being executed
Command string `json:"command"`
// Subcommands is the list of commands queued when a transaction starts, eg: by using the MULTI command
Subcommands []string `json:"subCommands,omitempty"`
Error string `json:"error,omitempty"`
// Error is the optional error that can be thrown by Redis when executing a command
Error string `json:"error,omitempty"`
}

func newRedisSpanTags(span *spanS) RedisSpanTags {
Expand Down

0 comments on commit 65938a1

Please sign in to comment.