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] Apply the date histogram rewrite optimization to range aggregation #14463

Merged
merged 1 commit into from
Jun 20, 2024

Commits on Jun 19, 2024

  1. Apply the date histogram rewrite optimization to range aggregation (#…

    …13865)
    
    * Refactor the ranges representation
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * Refactor try fast filter
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * Main work finished; left the handling of different numeric data types
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * buildRanges accepts field type
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * first working draft probably
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * add change log
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * accommodate geo distance agg
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * Fix test
    
    support all numeric types
    minus one on the upper range
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * [Refactor] range is lower inclusive, right exclusive
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * adding test
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * Adding test and refactor
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * refactor
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * add test
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * add test and update the compare logic in tree traversal
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * fix test, add random test
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * refactor to address comments
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * small potential performance update
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * fix precommit
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * refactor
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * refactor
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * set refresh_interval to -1
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * address comment
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * address comment
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * address comment
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    * Fix test
    
    To understand fully about the double and bigdecimal usage in scaled float field will take more time.
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    
    ---------
    
    Signed-off-by: bowenlan-amzn <[email protected]>
    (cherry picked from commit 57fb50b)
    Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
    github-actions[bot] committed Jun 19, 2024
    Configuration menu
    Copy the full SHA
    a9c175d View commit details
    Browse the repository at this point in the history