Skip to content

Conversation

heaths
Copy link
Member

@heaths heaths commented Oct 14, 2025

No description provided.

@Copilot Copilot AI review requested due to automatic review settings October 14, 2025 00:20
@github-actions github-actions bot added the Azure.Core The azure_core crate label Oct 14, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a new benchmark to measure URL construction performance in the Azure Core SDK. The benchmark compares two different approaches for building URLs with path parameters and query strings.

  • Adds a new url_construction benchmark function with two test cases
  • Compares "append" method (using multiple join() calls) vs "format" method (using string replacement)
  • Integrates the new benchmark into the existing criterion benchmark suite

@heaths
Copy link
Member Author

heaths commented Oct 14, 2025

@jhendrixMSFT @antkmsft it's a little faster to format paths like .NET is doing vs. formatting:

image

/cc @LarryOsterman @RickWinter

@heaths heaths merged commit d21d2e6 into Azure:main Oct 14, 2025
20 checks passed
@heaths heaths deleted the url-benchmarks branch October 14, 2025 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Azure.Core The azure_core crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants