test: add benchmark coverage for ExecutorEventHandler performance#6344
Open
sachin9058 wants to merge 1 commit intomindersec:mainfrom
Open
test: add benchmark coverage for ExecutorEventHandler performance#6344sachin9058 wants to merge 1 commit intomindersec:mainfrom
sachin9058 wants to merge 1 commit intomindersec:mainfrom
Conversation
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
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.
Summary
This PR adds benchmark coverage for
ExecutorEventHandler.HandleEntityEventto evaluate its performance characteristics.Currently, there is no benchmark for this critical execution path, making it difficult to understand throughput and memory allocation behavior or detect performance regressions over time. This change introduces both sequential and parallel benchmarks to provide a baseline for future performance analysis.
The benchmarks use a minimal stub executor to isolate handler logic and avoid external dependencies.
No additional dependencies are required for this change.
Fixes #6343
Testing
go test ./internal/engine -bench=.
Benchmark results:
BenchmarkHandleEntityEvent-12 1803787 635.4 ns/op 840 B/op 14 allocs/op
BenchmarkHandleEntityEventParallel-12 2365395 499.8 ns/op 838 B/op 14 allocs/op
The parallel benchmark shows improved throughput compared to sequential execution, indicating the handler scales well under concurrent workloads.
No functional changes were introduced, as this PR only adds benchmark tests.
No functional changes were introduced, as this PR only adds benchmark tests.