You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucenenet.apache.org by GitBox <gi...@apache.org> on 2022/05/22 14:27:11 UTC

[GitHub] [lucenenet] NightOwl888 opened a new pull request, #636: Lucene.Net.Analysis.Fa/PersianAnalyzer: Removed PersianStemmer as per apache/lucene#904

NightOwl888 opened a new pull request, #636:
URL: https://github.com/apache/lucenenet/pull/636

   Reverted changes from #571 as was done in apache/lucene#904. Changed `TestPersianStemFilter` to use mocks.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@lucenenet.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [lucenenet] NightOwl888 merged pull request #636: Lucene.Net.Analysis.Fa/PersianAnalyzer: Removed PersianStemmer as per apache/lucene#904

Posted by GitBox <gi...@apache.org>.
NightOwl888 merged PR #636:
URL: https://github.com/apache/lucenenet/pull/636


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@lucenenet.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [lucenenet] mocobeta commented on pull request #636: Lucene.Net.Analysis.Fa/PersianAnalyzer: Removed PersianStemmer as per apache/lucene#904

Posted by GitBox <gi...@apache.org>.
mocobeta commented on PR #636:
URL: https://github.com/apache/lucenenet/pull/636#issuecomment-1139781543

   We still have `PersianStemmer` in `PersianAnalyzer` on the `main` branch, it's a great addition - on the other hand, we have to consider backward compatibility on a major release (not to force users to reindex the whole data) so we can't expose the new stemmer for the default analyzer in 9.x. We will definitely have it in PerianAnalyzer in Lucene 10.0...


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@lucenenet.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org