Skip to content

SOLR-13138: remove deprecated LegacyBM25SimilarityFactory#274

Merged
cpoerschke merged 7 commits intoapache:mainfrom
cpoerschke:SOLR-13138-LegacyBM25Similarity-remove
Sep 29, 2021
Merged

SOLR-13138: remove deprecated LegacyBM25SimilarityFactory#274
cpoerschke merged 7 commits intoapache:mainfrom
cpoerschke:SOLR-13138-LegacyBM25Similarity-remove

Conversation

@cpoerschke
Copy link
Copy Markdown
Contributor

https://issues.apache.org/jira/browse/SOLR-13138

…Similarity-remove

Resolved Conflicts:
	solr/core/src/java/org/apache/solr/search/similarities/BM25SimilarityFactory.java
	solr/core/src/java/org/apache/solr/search/similarities/LegacyBM25SimilarityFactory.java
	solr/solr-ref-guide/src/schema-elements.adoc
@cpoerschke cpoerschke marked this pull request as ready for review August 29, 2021 17:36
Copy link
Copy Markdown
Contributor

@janhoy janhoy left a comment

Choose a reason for hiding this comment

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

LGTM, after a quick review. Have not had chance to compile or do thorough review, look for other mentions etc, so feel free to commit if you feel confident enough. Or else I’ll give it another view next week.

@cpoerschke
Copy link
Copy Markdown
Contributor Author

  • merged in latest origin/main
  • precommit and tests pass
  • git grep LegacyBM25Similarity re-checked for additional mentions (none found)

Will proceed to merge next then. And as always, happy to action any post-commit review feedback.

@cpoerschke cpoerschke merged commit 3a5e8ce into apache:main Sep 29, 2021
@cpoerschke cpoerschke deleted the SOLR-13138-LegacyBM25Similarity-remove branch September 29, 2021 08:39
janhoy pushed a commit to cominvent/solr that referenced this pull request Sep 29, 2021
epugh pushed a commit to epugh/solr that referenced this pull request Oct 22, 2021
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.

2 participants