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] Improve performance of encoding composite keys in multi-term aggregations #9434

Merged
merged 1 commit into from
Aug 18, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 46f2bd0 from #9412.

…ions (#9412)

* Improve performance of encoding composite keys in multi-term aggregations

Signed-off-by: Ketan Verma <[email protected]>

* Rename StreamOutput::getGenericValueWriter to getGenericValueWriterByClass

Signed-off-by: Ketan Verma <[email protected]>

* Rename StreamOutput::getGenericValueWriterByClass to getWriter and remove unused code

Signed-off-by: Ketan Verma <[email protected]>

---------

Signed-off-by: Ketan Verma <[email protected]>
(cherry picked from commit 46f2bd0)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@opensearch-trigger-bot
Copy link
Contributor Author

Compatibility status:

Checks if related components are compatible with change 2b6fa05

Incompatible components

Incompatible components: [https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/performance-analyzer.git]

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/ml-commons.git]

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov
Copy link

codecov bot commented Aug 18, 2023

Codecov Report

Merging #9434 (c7eddca) into 2.x (2b6fa05) will decrease coverage by 0.01%.
The diff coverage is 94.64%.

@@             Coverage Diff              @@
##                2.x    #9434      +/-   ##
============================================
- Coverage     70.90%   70.89%   -0.01%     
- Complexity    57628    57651      +23     
============================================
  Files          4762     4762              
  Lines        272064   272081      +17     
  Branches      40111    40110       -1     
============================================
+ Hits         192895   192898       +3     
- Misses        62636    62655      +19     
+ Partials      16533    16528       -5     
Files Changed Coverage Δ
...ggregations/bucket/terms/MultiTermsAggregator.java 82.07% <93.75%> (+2.88%) ⬆️
...opensearch/core/common/io/stream/StreamOutput.java 94.56% <100.00%> (+0.01%) ⬆️

... and 446 files with indirect coverage changes

@reta reta merged commit b68d6ff into 2.x Aug 18, 2023
@github-actions github-actions bot deleted the backport/backport-9412-to-2.x branch August 18, 2023 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant