[Port #6294] add simple actor telemetry #6299
Merged
Conversation
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
* added initial actor telemetry for akkadotnet#6293 * added basic telemetry tests for local actors * added spec to validate that `RemoteActorRef` doesn't influence counters * updated `SpawnActorBenchmarks` to include telemetry impact * converted telemetry events into `sealed class`es with `internal` constructors * removed `Reason` (cherry picked from commit 7f68c48)
Arkatufus
changed the title
|
@Arkatufus looks like all of the API approvals barfed here |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Port of Port #6294
added initial actor telemetry for Telemetry: publish events for each actor create / restart / terminate #6293
added basic telemetry tests for local actors
added spec to validate that
RemoteActorRefdoesn't influence countersupdated
SpawnActorBenchmarksto include telemetry impactconverted telemetry events into
sealed classes withinternalconstructorsremoved
Reason(cheery-picked from 7f68c48)