-
Notifications
You must be signed in to change notification settings - Fork 482
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add spanset attributes #2478
Add spanset attributes #2478
Conversation
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
This reverts commit d5eeb74.
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
case TypeStatus: | ||
if s.Status > other.Status { | ||
return 1 | ||
} else if s.Status < other.Status { | ||
return -1 | ||
} | ||
case TypeKind: | ||
if s.Kind > other.Kind { | ||
return 1 | ||
} else if s.Kind < other.Kind { | ||
return -1 | ||
} |
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.
Does this comparison make sense for status
and kind
? What does a status bigger than another mean?
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.
I agree that it's strange and one status isn't really comparable to another status in this way. The only reason I included them was so that if we were to sort Statics like Kind/Status using this function the results would be deterministic.
What this PR does:
This PR does some cleanup in preparation for passing arbitrary attributes through the Fetch method.
In order to keep these PRs more easily reviewable I'm going to cut this off here. Next I will work on actually preserving these through the fetch layer.
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]