Skip to content

add formatter - #10

Merged
rochdev merged 1 commit into
masterfrom
formatter
Feb 5, 2018
Merged

rochdev merged 1 commit into
masterfrom
formatter

Conversation

@rochdev

@rochdev rochdev commented Feb 2, 2018

Copy link
Copy Markdown
Member

Adds a formatter that converts an OpenTracing span to a Datadog trace span.

@palazzem palazzem added the core label Feb 5, 2018
@palazzem palazzem added this to the 0.1.0 milestone Feb 5, 2018
Comment thread src/format.js
const map = {
'service.name': 'service',
'span.type': 'type',
'resource.name': 'resource'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

Comment thread src/format.js
}
}

function extractTags (trace, tags) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this approach works, and probably we should do something similar to report errors (from OpenTracing API). We can chat about that later and not in this PR since attaching errors is something we can do later.

@rochdev
rochdev merged commit a3a6afc into master Feb 5, 2018
@rochdev
rochdev deleted the formatter branch February 5, 2018 22:41
alrosot pushed a commit to alrosot/dd-trace-js that referenced this pull request May 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants