Skip to content
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 new input/output type for get operations & util function to read span kind tag #1931

Merged
merged 2 commits into from
Nov 19, 2019

Conversation

guo0693
Copy link
Contributor

@guo0693 guo0693 commented Nov 18, 2019

Signed-off-by: Jun Guo [email protected]

Which problem is this PR solving?

Short description of the changes

Per #1924 (review), I am going to open a few small PRs to replace #1924

  • Add new input/output type for get operatoins
  • Add util function to read span kind tag

model/span.go Outdated Show resolved Hide resolved
model/span.go Outdated Show resolved Hide resolved
@guo0693 guo0693 changed the title #1921 1. Add new input/output type for get operatoins #1921 1. Add new input/output type for get operations & util function to read span kind tag Nov 18, 2019
@guo0693 guo0693 force-pushed the #1921-1 branch 3 times, most recently from a1ddd30 to 4445bd2 Compare November 19, 2019 16:19
@codecov
Copy link

codecov bot commented Nov 19, 2019

Codecov Report

Merging #1931 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1931      +/-   ##
==========================================
+ Coverage   98.45%   98.46%   +<.01%     
==========================================
  Files         198      198              
  Lines        9740     9744       +4     
==========================================
+ Hits         9590     9594       +4     
  Misses        114      114              
  Partials       36       36
Impacted Files Coverage Δ
model/span.go 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 82e182c...818c0bc. Read the comment docs.

@yurishkuro yurishkuro merged commit b467041 into jaegertracing:master Nov 19, 2019
@guo0693 guo0693 deleted the #1921-1 branch November 19, 2019 18:16
@pavolloffay pavolloffay changed the title #1921 1. Add new input/output type for get operations & util function to read span kind tag Add new input/output type for get operations & util function to read span kind tag Dec 17, 2019
@pavolloffay pavolloffay added this to the Release 1.16 milestone Dec 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants