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] [BUG] Flaky search.aggregation/230_composite_unsigned tests due to nested aggregations order #7587

Merged
merged 1 commit into from
May 16, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport af78ef9 from #7548.

…sted aggregations order (#7548)

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
(cherry picked from commit af78ef9)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov
Copy link

codecov bot commented May 16, 2023

Codecov Report

Merging #7587 (7511ece) into 2.x (1dbdf76) will increase coverage by 0.03%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                2.x    #7587      +/-   ##
============================================
+ Coverage     70.31%   70.34%   +0.03%     
- Complexity    59937    59981      +44     
============================================
  Files          4882     4882              
  Lines        288376   288439      +63     
  Branches      41920    41931      +11     
============================================
+ Hits         202777   202908     +131     
+ Misses        68613    68540      -73     
- Partials      16986    16991       +5     
Impacted Files Coverage Δ
...ggregations/bucket/composite/LongValuesSource.java 92.59% <ø> (ø)
...ons/bucket/composite/UnsignedLongValuesSource.java 0.00% <0.00%> (ø)

... and 471 files with indirect coverage changes

@dblock dblock merged commit 6aecbb2 into 2.x May 16, 2023
@github-actions github-actions bot deleted the backport/backport-7548-to-2.x branch May 16, 2023 19:41
andrross added a commit that referenced this pull request May 16, 2023
…aints (after update to Jackson 2.15.0 and above) (#7584)

* Provide mechanism to configure XContent parsing constraints (after update to Jackson 2.15.0 and above) (#7550)

* Provide mechanism to configure XContent parsing constraints (after update to Jackson 2.15.0 and above)

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

* Address code review comments

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

---------

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
(cherry picked from commit 03e41e3)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* Add changelog entry from #7587

Signed-off-by: Andrew Ross <andrross@amazon.com>

* Lift the XContent parsing constraint limits to preserve backward compatibility

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

---------

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Signed-off-by: Andrew Ross <andrross@amazon.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Andrew Ross <andrross@amazon.com>
Co-authored-by: Andriy Redko <andriy.redko@aiven.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants