-
Notifications
You must be signed in to change notification settings - Fork 320
Distributed Tracing for Entities (Isolated) #1198
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
Merged
sophiatev
merged 71 commits into
main
from
stevosyan/distributed-tracing-for-entities-isolated
May 20, 2025
Merged
Changes from all commits
Commits
Show all changes
71 commits
Select commit
Hold shift + click to select a range
af21cb2
added distributed tracing for entities
8846d70
change in how the distributed trace context is propagated in the case…
d3f25f3
slight style update
ed72bf7
some more tiny style things
1c271de
addressing two more PR comments
ffe061e
forgot to add one annotation
7462c40
yet another small style change
8871873
addressing PR comments
d49c005
forgot one file
7175737
addressing more PR comments
b2f859f
forgot two comments
a904e3e
first commit!
869d1a2
addressing a few small comments
25b87ad
added some new lines back
34a09f1
added comments, fixed up some incorrect logic regarding linking trace…
129a2e6
trying to add support for tracing when ShimDurableTaskClient is used …
3116463
refactored most of the tracing into this repo to more accurately refl…
87776bc
fixing spacing
97cd8e6
slight change in formatting of the create orchestration trace
9ed4bf6
aligning the isolated case with the new in-process changes
864a068
pushing what i have so far
8e1ff2e
seems like everything is working and aligned with the new in-process …
900567e
adding back new lines at the end of files
f71e300
missed some
d01179f
addressing PR comments
30db2fb
adding back new lines at the ends of files
c4dc784
trying to fix these line endings
41b0280
dealing with new lines again
dcac8a8
tiny change
f698251
Merge branch 'stevosyan/distributed-tracing-for-entities' into stevos…
2bd49c7
attempting to fix some whitespace issues
d82f3a4
attempting to fix some whitespace issues
128970b
attempting to fix some whitespace issues
9a54fef
missed a null check
82bc84d
added a check for if we cannot successfully parse the parent trace co…
2be54cf
moving all trace activities into TaskEntityDispatcher to avoid any ex…
9aec0fc
reverting to old design
d8c10bb
missed some
6b2bce1
and missed some more
0b56873
missed even more
ac98d23
will it ever end
9ef781c
last one i think
8e203b0
Merge branch 'stevosyan/distributed-tracing-for-entities' into stevos…
ee86dd2
moved activity for entity starting an orchestration back into webjobs
2206569
as always missed some stuff
d09e145
Merge branch 'stevosyan/distributed-tracing-for-entities' into stevos…
5f9842c
missed a spacing thing
a22ba16
fixing a bug where we needed a parent trace context for orchestration…
ba75851
added support for an entities enabled flag
87703e6
Merge branch 'stevosyan/distributed-tracing-for-entities' into stevos…
14faf90
added a start time to OperationResult
92e5c26
added a null check for client span ID when creating the activity for …
37764a6
Merge branch 'stevosyan/distributed-tracing-for-entities' into stevos…
13d56ad
getting rid of an extra line
1d6571b
Merge branch 'stevosyan/distributed-tracing-for-entities' into stevos…
9ad0099
addressing PR comment
389711f
Merge branch 'stevosyan/distributed-tracing-for-entities' into stevos…
69d968e
Merge branch 'main' into stevosyan/distributed-tracing-for-entities
e172546
addressing a few PR comments
b3405d3
Merge branch 'stevosyan/distributed-tracing-for-entities' into stevos…
5062030
addressing some PR comments, slightly changing the logic of the modif…
a97b5fe
tiny style update
0994ec5
missed a line
3ee6b03
slight style update... decided not to unnecessarily check for a inval…
2785860
Merge branch 'stevosyan/distributed-tracing-for-entities' into stevos…
b4054f5
addressing PR comments
8f4171b
fix for the bug with the scheduled time - if the scheduled time for a…
c22275b
tiny method name change
0d30abf
addressing PR comments
71d0f06
changed startTime/endTime to startTimeUtc/endTimeUtc and made the req…
18f8aeb
Merge branch 'main' into stevosyan/distributed-tracing-for-entities-i…
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
There are no files selected for viewing
This file contains hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.