-
Notifications
You must be signed in to change notification settings - Fork 73
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
improvements for PR #1 #2
Merged
Merged
Changes from 1 commit
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
e6392b6
Initial commit
dawallin 26ce53e
Changed License to MIT
dawallin 05e3379
.gitignore for visual studio project
dawallin 5a7a752
Readme file added.
dawallin 1a956bc
OpenTracing initial commit
dawallin e096e40
BasicTracer initial commit
dawallin f80f893
HttpClientCarrair initial commit
dawallin c2df056
Build status in readme
dawallin e0939a1
Removed SpanFactory
dawallin a5334d4
Removed SpanRecorder
dawallin 9571514
Renamed BasicTracerContext to OpenTracingSpanContext (from BasicSpanC…
dawallin b90961a
Moved Tracer and Span implementations to BasicTracer (from OpenTracin…
dawallin d570a6a
Removed leaking context in StartSpanOptions.Parent.
dawallin b4387e5
Moved the OpenTracing Tracer and Span to root namespace.
dawallin 6b0eb32
Moved the BasicTracer Tracer and Span to root namespace
dawallin 8ff3c6c
Fixed unit test name (from copy paste typo error).
dawallin 583e0df
Changed assembly name to simply OpenTracing.dll
dawallin 668dfd4
Nuspec:s for OpenTracing and BasicTracer
dawallin af55c33
Clean up some namespaces.
dawallin 5d4c4b9
Removed many generic type constraints on interfaces not needed.
dawallin 68e7316
Moved TracerBuilder to BasicTracer
dawallin 72a26c4
Moved SpanContext interfaces out from OpenTracing into BasicTracer.
dawallin 2951da2
Simplified ContextMapper and removed TextMapFormat
dawallin 97de99e
Merge branch 'InitialPR' into master
dawallin 6656848
build script; rename to src (to better follow new Microsoft repos)
cwe1ss c253d97
since parent folder is now called "src", library folder can be "OpenT…
cwe1ss 7189a71
nuget.exe not necessary with dotnet cli
cwe1ss 094de2c
converted netstandard
cwe1ss 435216f
refactoring
cwe1ss 1e2352d
refactoring
cwe1ss ce43aa5
abstract base class for Span, singleton for NullTracer
cwe1ss d2d8ea8
SetTag now accepts an object
cwe1ss 3f573ca
built-in tags are now regular ISpan extensions; BasicTracer compiles
cwe1ss 705dec5
API refinements
cwe1ss 1e6e7e2
tests succeed
cwe1ss ab72020
separate files for interfaces
cwe1ss 62a4bdc
move tests to test folder
cwe1ss e532779
renamed IntegrationTests project to just tests
cwe1ss 3f4c0b3
moved baggage dictionary into its own class
cwe1ss 65885ee
removed SpanContext.GetBaggageItems() from interface
cwe1ss b79bb3c
naming etc
cwe1ss dc9c8fd
visual studio files
cwe1ss a29b4bd
stuff that accepts ISpanContext now also accepts ISpan
cwe1ss 317f958
accidentally commited sample code; fixed powershell warnings
cwe1ss 5db8b2d
PoC for AspNetCore integration
cwe1ss e052b65
big refactoring after gitter discussion; docs
cwe1ss 5ea4551
renamed Log() to LogEvent() as of https://github.com/opentracing/open…
cwe1ss 7d2cca2
TODOs
cwe1ss d39266a
removed get/setBaggageItem from SpanContext - as of opentracing.githu…
cwe1ss f4f1488
SpanBuilder from @dawallin
cwe1ss 8c25747
SetOperationName(), BasicTracer: removed baggage key restrictions
cwe1ss e9c8ae8
Format/Carrier now almost equal to Java implementation
cwe1ss f55fdd6
AspNetCore is now in a separate branch - will move to a different rep…
cwe1ss File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
renamed Log() to LogEvent() as of opentracing/opentracing.io#26
- Loading branch information
commit 5ea45519461f6c61e6ed8c9e3ccb1159b8d2c6b9
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just FYI, logging is a bit of a moving target right now... I'm going to make some progress on opentracing/opentracing.io#96 today or this weekend.