feat(client): improve perfomance of generating path for client#2046
feat(client): improve perfomance of generating path for client#2046sterligov wants to merge 2 commits into
Conversation
Kusari Analysis ResultsAnalysis for commit: b690045, performed at: 2025-08-02T11:55:19Z • • Recommendation✅ PROCEED with this Pull Request Summary✅ No Flagged Issues Detected All values appear to be within acceptable risk parameters. No pinned version dependency changes, code issues or exposed secrets detected! Found this helpful? Give it a 👍 or 👎 reaction! |
501b237 to
ea62ed3
Compare
|
Kusari PR Analysis rerun based on - ea62ed3 performed at: 2025-08-02T05:09:15Z - link to updated analysis |
ea62ed3 to
fab1963
Compare
|
Kusari PR Analysis rerun based on - fab1963 performed at: 2025-08-02T05:11:59Z - link to updated analysis |
fab1963 to
a3cd398
Compare
|
Kusari PR Analysis rerun based on - a3cd398 performed at: 2025-08-02T06:16:38Z - link to updated analysis |
a3cd398 to
1511442
Compare
|
Kusari PR Analysis rerun based on - 1511442 performed at: 2025-08-02T07:22:05Z - link to updated analysis |
|
Thanks, will try and review at some point. Is it worth committing the benchmark code for future testing? |
|
Kusari PR Analysis rerun based on - 3acf76d performed at: 2025-08-02T11:53:02Z - link to updated analysis |
I added a commit with a benchmark test |
3acf76d to
b690045
Compare
|
Kusari PR Analysis rerun based on - b690045 performed at: 2025-08-02T11:55:19Z - link to updated analysis |
|
Thanks. From what I can see: Before: After: Is that right? The performance improvement seems fairly low |
In this pull request, I made a small performance improvement to the client by using string concatenation instead of fmt.Sprintf. Below, I’ll include the benchmark results.