Skip to content

Removing all deprecated TopScoreDocCollector + TopFieldCollector methods (#create, #createSharedManager)#13617

Merged
javanna merged 1 commit intoapache:mainfrom
slow-J:remove_deprecated
Jul 30, 2024
Merged

Removing all deprecated TopScoreDocCollector + TopFieldCollector methods (#create, #createSharedManager)#13617
javanna merged 1 commit intoapache:mainfrom
slow-J:remove_deprecated

Conversation

@slow-J
Copy link
Contributor

@slow-J slow-J commented Jul 29, 2024

Following up on: #13500

These methods already marked for deprecation in 9.x and we previously removed all internal use of these methods in 10.0 (in #13500).

Tested with: ./gradlew tidy && ./gradlew test

Closes: #13499

…ods (#create, #createSharedManager)

These are already marked for deprecation in 9.x and we previously removed all internal use of these methods in 10.0.
Copy link
Contributor

@javanna javanna left a comment

Choose a reason for hiding this comment

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

Looks great, thank you!

@javanna javanna added this to the 10.0.0 milestone Jul 30, 2024
@javanna javanna merged commit 74865bb into apache:main Jul 30, 2024
@javanna
Copy link
Contributor

javanna commented Jul 30, 2024

Thanks again @slow-J . Heads up, I moved the changes entry to the "API changes" section above after merging, I did not catch this while reviewing :)

@slow-J
Copy link
Contributor Author

slow-J commented Jul 30, 2024

My bad, sorry for the mistake, thanks for catching it and the review!

@slow-J slow-J deleted the remove_deprecated branch July 30, 2024 10:24
@javanna
Copy link
Contributor

javanna commented Jul 30, 2024

My bad, sorry for the mistake, thanks for catching it and the review!

no big deal, just a small oversight. Cheers!

javanna added a commit to elastic/elasticsearch that referenced this pull request Aug 28, 2024
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.

Remove the @Deprecated methods from TopScoreDocCollector and TopFieldCollector

2 participants