Skip to content

Reduce BWC version for HotThreads.sort #79583

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

Merged

Conversation

grcevski
Copy link
Contributor

The sort by change has been merged in 7.x now and therefore
we should make 8.0 use the same BWC version.

Relates to #79392 and #79507

The sort by change has been merged in 7.x now and therefore
we should make 8.0 use the same BWC version.

Relates to elastic#79392 and elastic#79507
@grcevski grcevski added >non-issue :Core/Infra/Core Core issues without another label v8.0.0 Team:Core/Infra Meta label for core/infra team labels Oct 20, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good, but are there no associated tests? I'm surprised CI has been passing with this out of sync.

@grcevski
Copy link
Contributor Author

I don't think there are any REST tests, let me see if we can add some.

@sethmlarson sethmlarson added the Team:Clients Meta label for clients team label Oct 21, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/clients-team (Team:Clients)

@grcevski grcevski merged commit 3f0ea76 into elastic:master Oct 25, 2021
grcevski added a commit to grcevski/elasticsearch that referenced this pull request Oct 26, 2021
The sort by change has been merged in 7.x now and therefore
we should make 8.0 use the same BWC version.

Relates to elastic#79392 and elastic#79507

Co-authored-by: Nhat Nguyen <nhat.nguyen@elastic.co>
grcevski added a commit that referenced this pull request Oct 27, 2021
Porting the tests and the updated rest spec to 7.16.


Co-authored-by: Nhat Nguyen <nhat.nguyen@elastic.co>
lockewritesdocs pushed a commit to lockewritesdocs/elasticsearch that referenced this pull request Oct 28, 2021
The sort by change has been merged in 7.x now and therefore
we should make 8.0 use the same BWC version.

Relates to elastic#79392 and elastic#79507

Co-authored-by: Nhat Nguyen <nhat.nguyen@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Core Core issues without another label >non-issue Team:Clients Meta label for clients team Team:Core/Infra Meta label for core/infra team v7.16.0 v8.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants