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

Data model #17

Merged
merged 4 commits into from
Apr 10, 2016
Merged

Data model #17

merged 4 commits into from
Apr 10, 2016

Conversation

chimericalidea
Copy link
Collaborator

a stab at converting the #11 into something helpful. Let me know if there are other "key insights" that we should highlight for the user. It doesn't seem to be helpful to explain to the user how we created the JSON, so I didn't explain that. If you disagree, we obviously can. :)

*** this includes the changes in #14 . do not merge unless you want to make everything live.

@abesto
Copy link
Member

abesto commented Apr 10, 2016

@adriancole how do you feel about this in its current state?
@chimericalidea since this PR was opened some extra information (even graphs!) was added to #11. Do you think it makes sense to incorporate some of those?

Spans must be transported from the services being traced to Zipkin collectors.
There are two primary transports, Scribe and Kafka. Scribe is deprecated.
Spans sent by the instrumented library must be transported from the services being traced to Zipkin collectors.
There are two primary transports: Scribe and Kafka. Scribe is shown in the figure. See [Span Receivers]({{ site.github.url }}/pages/span_receivers) for more information.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

HTTP is also a primary transport

@codefromthecrypt
Copy link
Member

LGTM thanks!

@abesto abesto merged commit adfbd2e into openzipkin:master Apr 10, 2016
abesto pushed a commit that referenced this pull request Jun 16, 2019
Adds sources download information for all projects except Brave
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.

3 participants