[Proposal] Use std-uritemplate
instead of maintaining a separate implementation
#7
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.
👋 Hi there, and thanks for the great project!
This is a proposal that I'd like to evaluate with this project maintainers.
In the Kiota project we faced the challenge of having to support implementations of uri-template in various languages, for this and more motivations we ended up rolling out a new one: std-uritemplate .
More recently I created a Quarkus-Kiota extension and alternative libraries, as expected, the provided Http Client implementation for the Quarkus integration is based on VertX 🙂 .
Now, having two independent implementations of the same (simple)algorithm on the classpath doesn't surely hurt much the users, but, merging the efforts has a couple of pretty interesting benefits:
With this PR I'm tentatively spearheading the collaboration opportunity, showing the tradeoffs of the approach in it's practical implementation.
Additional notes:
talking about performance, in the early days of
std-uritemplate
I have been assisted by @franz1981 and I just updated a quick JMH benchmark to demonstrate that the two implementation have comparable results.Happy to hear your thoughts!