Skip to content
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

[Backport] [2.x] [Concurrent Segment Search]: Add support for query profiler with concurrent aggregation (#9248) #9835

Closed

Commits on Sep 6, 2023

  1. Add support for query profiler with concurrent aggregation (opensearc…

    …h-project#9248)
    
    * Add support for query profiler with concurrent aggregation (opensearch-project#9248)
    
    Signed-off-by: Ticheng Lin <[email protected]>
    
    * Refactor and work on the PR comments
    
    Signed-off-by: Ticheng Lin <[email protected]>
    
    * Update collectorToLeaves mapping for children breakdowns post profile metric collection and before creating the results
    
    Signed-off-by: Sorabh Hamirwasia <[email protected]>
    
    * Refactor logic to compute the slice level breakdown stats and query level breakdown stats for concurrent search case
    
    Signed-off-by: Sorabh Hamirwasia <[email protected]>
    
    * Fix QueryProfilePhaseTests and QueryProfileTests, and parameterize QueryProfilerIT with concurrent search enabled
    
    Signed-off-by: Ticheng Lin <[email protected]>
    
    * Handle the case when there are no leaf context to compute the profile stats to return default stats
    for all breakdown type along with min/max/avg values. Replace queryStart and queryEnd time with queryNodeTime
    
    Signed-off-by: Sorabh Hamirwasia <[email protected]>
    
    * Add UTs for ConcurrentQueryProfileBreakdown
    
    Signed-off-by: Sorabh Hamirwasia <[email protected]>
    
    * Add concurrent search stats test into the QueryProfilerIT
    
    Signed-off-by: Ticheng Lin <[email protected]>
    
    * Address review comments
    
    Signed-off-by: Sorabh Hamirwasia <[email protected]>
    
    ---------
    
    Signed-off-by: Ticheng Lin <[email protected]>
    Signed-off-by: Sorabh Hamirwasia <[email protected]>
    Co-authored-by: Sorabh Hamirwasia <[email protected]>
    Signed-off-by: Ticheng Lin <[email protected]>
    ticheng-aws and sohami committed Sep 6, 2023
    Configuration menu
    Copy the full SHA
    5f1c7c4 View commit details
    Browse the repository at this point in the history