resources map to jaeger process tags

[danrusei]

#205
I hope my understanding about what is requested here is correct, otherwise I'll modify based on the feedback.

[linux-foundation-easycla]

The committers are authorized under a signed CLA.

• ✅ Dan Rusei (fbbcb90)

[TommyCpp]

Good start, thanks! have some suggestions

[TommyCpp]

Sorry for the late reply. LGTM. Thanks!

[jtescher]

@danrusei generally looks good, just want to make sure this won't add duplicate process tags when exporting batches of spans

[danrusei]

@danrusei generally looks good, just want to make sure this won't add duplicate process tags when exporting batches of spans

thank you, would this work ?

for tag in build_process_tags(&span) {
                    if !process_tags.contains(&tag) {
                        process_tags.push(tag)
                    }
                }                

[jtescher]

@danrusei That should work but it may do a lot of work checking each time. The resource should be the same for all spans, so you could do something like for (idx, span) in spans.enumerate() { if idx == 0 { add tags} ... } to only add them the first time?

[jtescher]

Looks good @danrusei, just have to rebase and make sure tests pass

[jtescher]

Looks good thanks @danrusei