-
Notifications
You must be signed in to change notification settings - Fork 13
Allow to extend Endpoint path #1008
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
e43f079
to
7db92f7
Compare
BenchmarksComparisonBenchmark execution time: 2025-04-15 14:43:36 Comparing candidate commit c2c152a in PR branch Found 1 performance improvements and 10 performance regressions! Performance is the same for 41 metrics, 2 unstable metrics. scenario:benching deserializing traces from msgpack to their internal representation
scenario:credit_card/is_card_number/ 3782-8224-6310-005
scenario:credit_card/is_card_number/ 378282246310005
scenario:credit_card/is_card_number/378282246310005
scenario:credit_card/is_card_number/x371413321323331
scenario:credit_card/is_card_number_no_luhn/x371413321323331
CandidateCandidate benchmark detailsGroup 1
Group 2
Group 3
Group 4
Group 5
Group 6
Group 7
Group 8
Group 9
Group 10
Group 11
Group 12
Group 13
BaselineOmitted due to size. |
05309bc
to
acf5797
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1008 +/- ##
==========================================
+ Coverage 71.48% 71.55% +0.07%
==========================================
Files 337 339 +2
Lines 50509 50727 +218
==========================================
+ Hits 36104 36298 +194
- Misses 14405 14429 +24
🚀 New features to boost your workflow:
|
Artifact Size Benchmark Reportaarch64-alpine-linux-musl
aarch64-unknown-linux-gnu
libdatadog-x64-windows
libdatadog-x86-windows
x86_64-alpine-linux-musl
x86_64-unknown-linux-gnu
|
I'm not entirely sure what you are after, since there's no elaboration beyond the title. The endpoint is roughly a scheme and authority. I think it wasn't meant to be a generic URL. It's basically "this is the server you talk to and how" but it doesn't know about URL paths and such. I'm not rejecting your PR, just trying to have a discussion about what the goal is. |
This is mostly a mix of several ideas and some experimentations. One thing I was trying to solve is to avoid recreating the endpoint when accessing a specific path (e.g. /v04/traces). There should be an easy way to send a request to a specific route of a server described by an endpoint. |
500dace
to
7db92f7
Compare
Replaces the add_path methods by Endpoint new path extension capability.
7db92f7
to
c2c152a
Compare
What API are you using to make the HTTP request? |
What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
How to test the change?
Describe here in detail how the change can be validated.