addiing parallelism to speed up performace issue #6111 - Githubissues

jaegertracing / jaeger

CNCF Jaeger, a Distributed Tracing Platform

https://www.jaegertracing.io/

Apache License 2.0

20.58k stars 2.45k forks source link

addiing parallelism to speed up performace issue #6111 #6139

Closed Insomniac2904 closed 3 weeks ago

Insomniac2904 commented 4 weeks ago

Which problem is this PR solving?

Resolves #6111

Description of the changes

Added t.parallel() in test functions and used goroutines for checking occurrences in LogMatcher function.

How was this change tested?

GOMAXPROCS=1 go test -parallel 128 -p 16 -json ./... | go run github.com/roblaszczak/vgt@latest

Checklist

[x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
[x] I have signed all commits
[ ] I have added unit tests for the new functionality
[x] I have run lint and test steps successfully
- for jaeger: make lint test
- for jaeger-ui: yarn lint and yarn test

Results

yurishkuro commented 3 weeks ago

Please read the ticket description carefully.