Skip to content
This repository was archived by the owner on Dec 23, 2023. It is now read-only.

Trace Benchmarks#1833

Merged
dinooliva merged 11 commits intocensus-instrumentation:masterfrom
dinooliva:trace-benchmarks
Apr 5, 2019
Merged

Trace Benchmarks#1833
dinooliva merged 11 commits intocensus-instrumentation:masterfrom
dinooliva:trace-benchmarks

Conversation

@dinooliva
Copy link
Copy Markdown
Contributor

Adds microbenchmarks for OpenCensus Trace.

@dinooliva dinooliva requested review from a team, rghetia and songy23 as code owners April 3, 2019 17:29
@dinooliva dinooliva changed the title Trace benchmarks Trace Benchmarks Apr 4, 2019
@Benchmark
@BenchmarkMode(Mode.AverageTime)
@OutputTimeUnit(TimeUnit.NANOSECONDS)
public byte[] encodeSpanBinary(Data data) {
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.

encode and decode will vary based on number of attributes, links and annotations. Will that be covered later?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Encoding/Decoding only apply to the span context - tracestate will get added eventually but we don't support that at the moment.

@dinooliva
Copy link
Copy Markdown
Contributor Author

Thanks!

@dinooliva dinooliva merged commit 221e480 into census-instrumentation:master Apr 5, 2019
@songy23 songy23 mentioned this pull request Apr 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants