- 
                Notifications
    You must be signed in to change notification settings 
- Fork 15
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
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.