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

Code improvements for ETW exporter #519

Merged
merged 94 commits into from
Apr 3, 2021
Merged
Changes from 1 commit
Commits
Show all changes
94 commits
Select commit Hold shift + click to select a range
6bd4aaa
Drafts for proper TLD operation. Verified that TLD flow works well wi…
maxgolov Jan 12, 2021
f8a3af1
Add CODEOWNERS for ETW exporter
maxgolov Jan 15, 2021
035d863
Add support for owning Properties container
maxgolov Jan 15, 2021
1141e29
Make SpanContextKeyValueIterable empty container by default
maxgolov Jan 15, 2021
01cd9ac
is_key_value_iterable does not property test for subclasses of KeyVal…
maxgolov Jan 15, 2021
01c693c
Allow C string type
maxgolov Jan 15, 2021
66b1ee2
Add ability for vendors to customize their ETW field names
maxgolov Jan 15, 2021
90ecfd0
Code refactor to address outstanding feature parity issues
maxgolov Jan 15, 2021
5acc2d3
Code formatting
maxgolov Jan 15, 2021
013d1fd
Fix glob for Bazel build
maxgolov Jan 15, 2021
956e38d
Fix Bazel formatting issue
maxgolov Jan 15, 2021
bcc54f7
Merge branch 'master' into maxgolov/etw_exporter
maxgolov Jan 15, 2021
846ad71
Wrong #ifdef in test
maxgolov Jan 16, 2021
16ccb2a
Add prometheus to Visual Studio 2019 CMake build configuration
maxgolov Jan 16, 2021
fc55f3c
Add support for span<uint8_t> and const char * to OTLP exporter
maxgolov Jan 16, 2021
1b039a8
Sort OwnedAttributeValue in the same order as AttributeValue (for con…
maxgolov Jan 16, 2021
af1718f
Use enums instead of hardcoded constants
maxgolov Jan 16, 2021
d142031
Add prometheus-cpp to vcpkg build on Windows
maxgolov Jan 16, 2021
c02a822
include(CTest) and only then if(BUILD_TESTING)
maxgolov Jan 16, 2021
d86267f
Merge branch 'maxgolov/etw_exporter' of https://github.com/open-telem…
maxgolov Jan 16, 2021
c4d73ad
Merge branch 'master' of https://github.com/open-telemetry/openteleme…
maxgolov Jan 16, 2021
afd035e
Fix CI failures
maxgolov Jan 16, 2021
a1141e4
Add nlohmann/json to CMake build
maxgolov Jan 16, 2021
93b2480
Compile empty stub for writeMsgPack when HAVE_MSGPACK is not defined
maxgolov Jan 16, 2021
897da33
Build Bazel with ETW-MsgPack exporter
maxgolov Jan 16, 2021
07952be
Add nlohmann_json to bazel build of ETW exporter
maxgolov Jan 16, 2021
09d8fa9
Merge branch 'master' of https://github.com/open-telemetry/openteleme…
maxgolov Jan 19, 2021
8a737c7
Incremental clean-up and simplify Bazel build
maxgolov Jan 20, 2021
a4b459d
Merge branch 'master' into maxgolov/etw_exporter
maxgolov Jan 20, 2021
0c3b167
Update .github/CODEOWNERS
maxgolov Jan 21, 2021
fc2cbb3
Merge branch 'master' into maxgolov/etw_exporter
maxgolov Jan 22, 2021
b7153eb
Allow populating Span from byte buffer
maxgolov Jan 29, 2021
a09a941
Allow populating TraceId from byte buffer
maxgolov Jan 29, 2021
5de423b
Allow populating ActivityId and RelatedActivity based on SpanId and P…
maxgolov Jan 29, 2021
4b5d3b7
Automated context propagation
maxgolov Jan 29, 2021
fdda4da
Include headers for CoCreateGuid
maxgolov Jan 29, 2021
ee14876
Added 3-level deep spans to test
maxgolov Jan 29, 2021
51461d4
Merge branch 'main' of https://github.com/open-telemetry/opentelemetr…
maxgolov Jan 29, 2021
ebcd88e
Merge branch 'main' of https://github.com/open-telemetry/opentelemetr…
maxgolov Feb 2, 2021
c81d08e
Merge branch 'main' of https://github.com/open-telemetry/opentelemetr…
maxgolov Feb 2, 2021
44f0ce2
Reformat code
maxgolov Feb 2, 2021
c3a7442
Merge branch 'main' of https://github.com/open-telemetry/opentelemetr…
maxgolov Feb 25, 2021
6d70f6d
Build Release without debug info
maxgolov Feb 25, 2021
81f0db9
Merge branch 'main' of https://github.com/open-telemetry/opentelemetr…
maxgolov Mar 11, 2021
9475c16
Rename test provider to OpenTelemetry-ETW-Provider
maxgolov Mar 11, 2021
0d993d0
Merge branch 'maxgolov/etw_exporter_ActivityId' of https://github.com…
maxgolov Mar 11, 2021
e159b2c
Reformat code using latest formatter
maxgolov Mar 11, 2021
4adafce
Merge branch 'main' of https://github.com/open-telemetry/opentelemetr…
maxgolov Mar 17, 2021
c9f1efa
define HAVE_TLD only if it hasn't been defined yet
maxgolov Mar 20, 2021
ec0ef2e
Fix C++17 issue regarding deprecated C++11 codecvt
maxgolov Mar 20, 2021
48adeea
Add tests for detailed decorator and min decorator
maxgolov Mar 20, 2021
c3d0dc8
Add smarter ref-counting logic to ETW provider, allowing multiple tra…
maxgolov Mar 20, 2021
753521b
ETW Tracer improvements:
maxgolov Mar 20, 2021
5ed213c
Code formatting changes
maxgolov Mar 20, 2021
1d46f76
Add time routines to utils
maxgolov Mar 23, 2021
131f801
Fix bug in format passing
maxgolov Mar 23, 2021
51bea36
Add separate tests for TLD (TraceLoggingDynamic) encoding and MsgPack…
maxgolov Mar 23, 2021
4543a47
Clean-up field names for ETW notation (common field names used for bo…
maxgolov Mar 23, 2021
48991c0
ETW/MsgPack: Chunk Span Start, Span Event, Span Stop as individual ET…
maxgolov Mar 23, 2021
8678a91
Merge branch 'main' of https://github.com/open-telemetry/opentelemetr…
maxgolov Mar 26, 2021
665422e
Fix GMock linkage error on Windows
maxgolov Mar 26, 2021
fb71a95
Merge branch 'maxgolov/cmake_windows_fix' of https://github.com/open-…
maxgolov Mar 26, 2021
f459bd3
Addressing code review comments
maxgolov Mar 27, 2021
4fd5528
Remove unused templates
maxgolov Mar 27, 2021
da56e3e
Add benchmark to compare Properties vs initializer list vs unordered_…
maxgolov Mar 30, 2021
2951a1e
Merge branch 'main' of https://github.com/open-telemetry/opentelemetr…
maxgolov Mar 30, 2021
127f149
Update exporters/etw/include/opentelemetry/exporters/etw/etw_provider.h
maxgolov Mar 31, 2021
b51040a
Merge branch 'main' of https://github.com/open-telemetry/opentelemetr…
maxgolov Mar 31, 2021
0f2b8d6
Merge branch 'main' of https://github.com/open-telemetry/opentelemetr…
maxgolov Apr 1, 2021
b78797f
Revert "Find GTest via CONFIG mode to reference gmock (#640)"
maxgolov Apr 1, 2021
a683ed5
Add time formatting utility
maxgolov Apr 1, 2021
1db003e
Rename 'kind' field to 'OpCode' (start or stop)
maxgolov Apr 1, 2021
3a7831a
Tests are no longer included in Windows build
maxgolov Apr 1, 2021
b0c31bf
New feature to consolidate all Span info on end of Span into separate…
maxgolov Apr 1, 2021
bcf7325
Apply code formatting rules
maxgolov Apr 1, 2021
1561bd0
Add support for const char* and byte arrays under feature gate in Zip…
maxgolov Apr 1, 2021
b51cd39
Rename type to Google coding style kTypeString
maxgolov Apr 2, 2021
61b2f7c
Addressing code review comments: remove api change
maxgolov Apr 2, 2021
467f823
Addressing code review comments: rename namespace from ETW to etw
maxgolov Apr 2, 2021
576d510
Remove ETW recordable class as it is not needed
maxgolov Apr 2, 2021
a1653b4
Remove HAVE_CSTRING_TYPE build option
maxgolov Apr 2, 2021
57ab55b
Remove API changes
maxgolov Apr 2, 2021
1b0baf7
Rename namespace from ETW to etw
maxgolov Apr 2, 2021
e37cee5
Rename namespace from ETW to etw
maxgolov Apr 2, 2021
63f793d
Address code review comment: rename enum types to follow Google codin…
maxgolov Apr 2, 2021
a8afc39
Remove HAVE_CSTRING_TYPE
maxgolov Apr 2, 2021
18b2f9b
Rename enum types to follow Google coding style
maxgolov Apr 2, 2021
c2777e2
Remove HAVE_CSTRING_TYPE build option
maxgolov Apr 2, 2021
bf8a3d4
Remove HAVE_CSTRING_TYPE option and rename enums to follow Google Cod…
maxgolov Apr 2, 2021
8feee53
Rename namespace from ETW to etw
maxgolov Apr 2, 2021
888a040
Rename types to match Google coding style
maxgolov Apr 2, 2021
69c5e66
Fix mismatched ifdef
maxgolov Apr 2, 2021
80512b5
Revert changes to TraceZ
maxgolov Apr 2, 2021
550f2d8
true should be returned from Null object pattern to indicate success
maxgolov Apr 2, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix mismatched ifdef
maxgolov committed Apr 2, 2021

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
commit 69c5e6681add527e5faa13065e3d5aea221e8756
2 changes: 1 addition & 1 deletion exporters/zipkin/src/recordable.cc
Original file line number Diff line number Diff line change
@@ -29,7 +29,7 @@ namespace zipkin
// Expecting to remove the two feature gates for:
// - HAVE_SPAN_BYTE - proposal for binary type or byte array (uint8_t[]).
//
#elif defined(HAVE_SPAN_BYTE)
#if defined(HAVE_SPAN_BYTE)
const int kAttributeValueSize = 15;
#else
const int kAttributeValueSize = 14;